FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

feat: Relax client-side validation for BigQuery entity IDs by jinseopkim0 · Pull Request #4000 · googleapis/java-bigquery · GitHub

This repository was archived by the owner on Mar 23, 2026. It is now read-only.
/ java-bigquery Public archive

feat: Relax client-side validation for BigQuery entity IDs - #4000

Merged
jinseopkim0 merged 2 commits into
mainfrom
fix-client-validation
Nov 20, 2025
Merged

feat: Relax client-side validation for BigQuery entity IDs#4000
jinseopkim0 merged 2 commits into
mainfrom
fix-client-validation

Conversation

Copy link
Copy Markdown
Contributor

Removes client-side checkArgument and checkNotNull validations from the constructors of TableId, DatasetId, RoutineId, ModelId, and ConnectionProperty.

This change makes the client library more robust by preventing crashes when the server sends a response with unexpected null or empty values in these fields.

b/460218010

product-auto-label Bot added size: s Pull request size is small. api: bigquery Issues related to the googleapis/java-bigquery API. labels Nov 14, 2025
jinseopkim0 force-pushed the fix-client-validation branch from b57a1f5 to 2190847 Compare November 14, 2025 19:14
Removes client-side `checkArgument` and `checkNotNull` validations from the constructors of `TableId`, `DatasetId`, `RoutineId`, `ModelId`, and `ConnectionProperty`.

This change makes the client library more robust by preventing crashes when the server sends a response with unexpected null or empty values in these fields.
jinseopkim0 force-pushed the fix-client-validation branch from 2190847 to db46518 Compare November 14, 2025 19:17
jinseopkim0 added kokoro:force-run Add this label to force Kokoro to re-run the tests. kokoro:run Add this label to force Kokoro to re-run the tests. labels Nov 20, 2025
yoshi-kokoro removed kokoro:run Add this label to force Kokoro to re-run the tests. kokoro:force-run Add this label to force Kokoro to re-run the tests. labels Nov 20, 2025
jinseopkim0 marked this pull request as ready for review November 20, 2025 11:03
jinseopkim0 requested review from a team and agrawal-siddharth November 20, 2025 11:03
jinseopkim0 merged commit c3548a2 into main Nov 20, 2025
23 of 24 checks passed
jinseopkim0 deleted the fix-client-validation branch November 20, 2025 20:59
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: bigquery Issues related to the googleapis/java-bigquery API. size: s Pull request size is small.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL