Skip to content

feat(bigquery): augment external data config#14397

Open
shollyman wants to merge 4 commits intogoogleapis:mainfrom
shollyman:fr-externaldata
Open

feat(bigquery): augment external data config#14397
shollyman wants to merge 4 commits intogoogleapis:mainfrom
shollyman:fr-externaldata

Conversation

@shollyman
Copy link
Copy Markdown
Contributor

@shollyman shollyman commented Apr 8, 2026

This is a PR for exposing additional fields present in ExternalDataConfiguration, namely controls for json handling, object metadata, and how uris are processed for some scenarios (explicit data files vs manifest files).

Testing exercises existing round tripping in unit tests, but does not introduce new integration testing.

#jetski

@product-auto-label product-auto-label bot added the api: bigquery Issues related to the BigQuery API. label Apr 8, 2026
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request introduces several new configuration options for BigQuery external data sources, including FileSetSpecType, JsonExtension, ObjectMetadata, and TimestampTargetPrecision. It also adds support for JsonOptions to handle character encoding for JSON data. Feedback focuses on adhering to Go naming conventions for acronyms (e.g., renaming 'Json' to 'JSON') and improving enum completeness by adding an 'Unspecified' value for FileSetSpecType.

Comment thread bigquery/external.go
Comment thread bigquery/external.go Outdated
Comment thread bigquery/external.go Outdated
Comment thread bigquery/external.go Outdated
Comment thread bigquery/external.go Outdated
Comment thread bigquery/external.go Outdated
Comment thread bigquery/external.go Outdated
@shollyman shollyman marked this pull request as ready for review April 9, 2026 17:08
@shollyman shollyman requested review from a team as code owners April 9, 2026 17:08
Copy link
Copy Markdown
Member

@quartzmo quartzmo left a comment

Choose a reason for hiding this comment

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

Great to get these new fields added to the client!

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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants