-
Notifications
You must be signed in to change notification settings - Fork 15
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
Support GEOGRAPHY types #7
Comments
I guess BigQuery added this type very recently (it is also in beta status), because I noticed it just last week. I definitely believe we do need to support it. Just converting I see it as some kind of "Loader enrichment" which transforms complex JSON types into DB's native, i.e. we configure that in
On the other hard we can add it to configuration of loader, but we're mixing concerns here: analysts should not ask DevOps to change configuration every time they've added new schema with geo data. Also, we don't want to restart the job just to reload this minor piece of configuration. Last option is to try to make Loader smart enough to figure out these columns by itself:
So far I prefer this option, its main disadvantage is that it is slightly implicit and magical. What if 3rd party schema encodes geo data with some completely unexpected properties, such as |
Agreed - it feels awkward putting logic that is essentially transformations into a JSON schema but you've raised the excellent point that haven't this hard coded in to the codebase makes this tricky to maintain from an analyst point of view. The magic option sounds desirable but it still feels like it needs some sort of high level configuration so whatever type inferences we are making don't incorrectly cast objects (whether that's GEOGRAPHY or another type in the future). Does having a system that allows UDFs make sense in terms of striking a balance between flexibility and configuration? e.g., something like the Javascript enrichment for transformation - though it wouldn't necessarily need to be Javascript. |
Given that BigQuery now has some geography support does it make sense to convert the latitude / longitude columns to a single
GEOGRAPHY
column of type ST_GEOGPOINT?The text was updated successfully, but these errors were encountered: