-
Notifications
You must be signed in to change notification settings - Fork 587
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Ah fix timestamp query #7319
Ah fix timestamp query #7319
Conversation
…n access use project and dataset in BQ query change disk size
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
1767a0a
to
7d9c703
Compare
|
||
LASTMODIFIED=$(bq show ~{"--project_id " + query_project} --location=US --format=json ${DATASET_TABLE_COLON} | python3 -c "import sys, json; print(json.load(sys.stdin)['lastModifiedTime']);") | ||
# bq needs the project name to be separate by a colon | ||
DATASET_TABLE_COLON=$(echo ~{fq_table} | sed 's/\./:/') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just to confirm... this only replaces the first instance of a .
correct, not all of them?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
correct. there is no '/g' at the end which means replace all of them.
tested on an extract of the 3 controls
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm. thanks for the changes
create .bigqueryrc to suppress warning messages
change disk size to 150