- Notifications
You must be signed in to change notification settings - Fork321
docs: add GEOGRAPHY data type code samples#428
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
These are added to a separate directory in order to isolate the GeoJSONand WKT dependencies from the other code samples.
snippet-botbot commentedDec 9, 2020 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Here is the summary of changes. You added 4 region tags.
This comment is generated bysnippet-bot.
|
| SELECT name, SUM(number) AS total_people | ||
| FROM `bigquery-public-data.usa_names.usa_1910_current` | ||
| GROUP BY name | ||
| SELECT COUNT(*) |
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.
Not really associated with this sample, but the system test flaked, so I figured I'd swap out this query with one from go/bq-long-queries
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.
Sounds good.
shollyman left a comment
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.
I was today years old when I found out that streaming inserts will accept either WKT or geojson as the field contents.
| SELECT name, SUM(number) AS total_people | ||
| FROM `bigquery-public-data.usa_names.usa_1910_current` | ||
| GROUP BY name | ||
| SELECT COUNT(*) |
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.
Sounds good.
| importnox | ||
| # WARNING - WARNING - WARNING - WARNING - WARNING |
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.
This file is generated?
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.
These are added to a separate directory in order to isolate the GeoJSON
and WKT dependencies from the other code samples.
Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:
Towards internal 171804665