Skip to content

bigquery: add setUseLegacySql to ViewDefinition#2743

Merged
pongad merged 1 commit intogoogleapis:masterfrom
pongad:bq-legacy
Jan 4, 2018
Merged

bigquery: add setUseLegacySql to ViewDefinition#2743
pongad merged 1 commit intogoogleapis:masterfrom
pongad:bq-legacy

Conversation

@pongad
Copy link
Copy Markdown
Contributor

@pongad pongad commented Jan 3, 2018

Fixes #2720.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jan 3, 2018
@tswast
Copy link
Copy Markdown
Contributor

tswast commented Jan 3, 2018

The test failure seems to be a problem loading plugins.

Copy link
Copy Markdown
Contributor

@tswast tswast left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

I notice you don't attempt to override the server default. That definitely keeps things simpler, but I do believe there was a requirement to make standard SQL the default. Do you want to follow-up with a PR to do that?

@pongad
Copy link
Copy Markdown
Contributor Author

pongad commented Jan 4, 2018

SGTM. I made #2748 so I don't forget.

@pongad pongad merged commit 75b9ab0 into googleapis:master Jan 4, 2018
@pongad pongad deleted the bq-legacy branch January 4, 2018 00:36
chingor13 pushed a commit that referenced this pull request Feb 20, 2026
This is to unblock protocol buffer version updates.
suztomo pushed a commit that referenced this pull request Mar 9, 2026
chingor13 pushed a commit that referenced this pull request Mar 30, 2026
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants