Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[BigQuery] Option to set useLegacySql to false on queries #1927

Closed
kelvinabrokwa opened this issue Jun 28, 2016 · 9 comments
Closed

[BigQuery] Option to set useLegacySql to false on queries #1927

kelvinabrokwa opened this issue Jun 28, 2016 · 9 comments
Assignees
@tseaver
Labels
api: bigquery Issues related to the BigQuery API. type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design.

Comments

@kelvinabrokwa
Copy link

kelvinabrokwa commented Jun 28, 2016 •
edited
Loading

My use case here is that there is a certain query that errors on legacy SQL (internal bug) and disabling it will allow the use of standard SQL which fixes it.

see https://cloud.google.com/bigquery/docs/reference/v2/jobs#resource-representations (find configuration.query.useLegacySql)

I'm happy to submit a PR is this option makes sense.
@tseaver tseaver added the api: bigquery Issues related to the BigQuery API. label Jun 28, 2016
@tseaver
Copy link
Contributor

tseaver commented Jun 28, 2016

@kelvinabrokwa Thanks for the report! We'd be glad to review / merge a PR for that feature.

@tseaver tseaver added the type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design. label Jun 28, 2016
@dhermes
Copy link
Contributor

dhermes commented Jun 29, 2016

@tseaver I can tackle this, sound good?

@dhermes dhermes self-assigned this Jun 29, 2016
@tseaver
Copy link
Contributor

tseaver commented Jun 29, 2016

@dhermes Sure, if @kelvinabrokwa doesn't already have it in progress.

@kelvinabrokwa
Copy link
Author

@dhermes go for it! 😄

@kelvinabrokwa
Copy link
Author

@dhermes any progress?

@dhermes
Copy link
Contributor

dhermes commented Jul 18, 2016

Doing it now. Sorry for the delay.

@kelvinabrokwa
Copy link
Author

@dhermes no worries! Thank you! 😄

@tseaver tseaver mentioned this issue Jul 18, 2016
Closed
@tseaver
Copy link
Contributor

tseaver commented Jul 18, 2016

Via #1992.

@tseaver tseaver closed this as completed Jul 18, 2016
@kelvinabrokwa
Copy link
Author

@dhermes @tseaver thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tseaver tseaver

Labels
api: bigquery Issues related to the BigQuery API. type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@tseaver @dhermes @kelvinabrokwa