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.

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

BigQuery query method should default to standard SQL syntax #791

Closed
tswast opened this issue Dec 14, 2017 · 8 comments
Closed

BigQuery query method should default to standard SQL syntax #791

tswast opened this issue Dec 14, 2017 · 8 comments
Assignees
Labels
api: bigquery Issues related to the BigQuery API. type: question Request for information or clarification. Not an issue.

Comments

@tswast
Copy link

tswast commented Dec 14, 2017

I believe this was a GA requirement from the BigQuery engineering team, that the default SQL syntax be changed at the client-library layer for GA. The backend API still defaults to legacy SQL, but they would like to discourage the use of legacy SQL in most applications.

@tswast tswast added the api: bigquery Issues related to the BigQuery API. label Dec 14, 2017
@jdpedrie
Copy link
Contributor

Hi @tswast! This was added in #640. It's described in our documentation here. (pardon the broken markdown, i just opened a PR to fix that).

@jdpedrie jdpedrie added the type: question Request for information or clarification. Not an issue. label Dec 14, 2017
@tswast
Copy link
Author

tswast commented Dec 14, 2017

Okay, thanks. I guess my problem was that https://github.com/GoogleCloudPlatform/php-docs-samples/tree/master/bigquery/shakespeare is using a very old version. Somehow I didn't end up with the latest version when I deleted the composer lock file and tried to reinstall.

Sorry for the noise!

@tswast tswast closed this as completed Dec 14, 2017
@tswast
Copy link
Author

tswast commented Dec 14, 2017

Oh, but I can't seem to install the latest version.

$ php composer.phar install
Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - The requested package google/cloud-bigquery ^0.46 exists as google/cloud-bigquery[dev-master, v0.1.0, v0.2.0, v0.2.1, v0.2.2, v0.2.3, v0.2.4, v0.3.0, v0.3.1] but these are rejected by your constraint.

My composer.json is

$ cat composer.json
{
    "require": {
        "google/cloud-bigquery": "^0.46"
    },
    "require-dev": {
        "phpunit/phpunit": "~4"
    }
}

That package name seems to match the installation instructions at https://github.com/GoogleCloudPlatform/google-cloud-php-bigquery

@tswast tswast reopened this Dec 14, 2017
@dwsupplee
Copy link
Contributor

0.46 is the version of the "umbrella" package which installs every component. When installing just BQ itself you will want to relax the version constraint down to 0.3.1

@tswast
Copy link
Author

tswast commented Dec 14, 2017

Ah, gotcha. Might be worth putting a badge on https://github.com/GoogleCloudPlatform/google-cloud-php-bigquery with the latest version?

@tswast tswast closed this as completed Dec 14, 2017
@jdpedrie
Copy link
Contributor

@tswast that's a great idea!

@tswast
Copy link
Author

tswast commented Dec 14, 2017

@jdpedrie
Copy link
Contributor

@tswast thanks for the PRs. However, those repos are Read-Only splits from this one! The file you're looking for is here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: bigquery Issues related to the BigQuery API. type: question Request for information or clarification. Not an issue.
Projects
None yet
Development

No branches or pull requests

3 participants