Skip to content

chore(bigquery-jdbc): fix validate getter tests to only test valid coercions - #14385

Merged
Neenu1995 merged 6 commits into
mainfrom
fix-test-null-conversion
Sep 15, 2026
Merged

Neenu1995 merged 6 commits into
mainfrom
fix-test-null-conversion

Conversation

@Neenu1995

Copy link
Copy Markdown
Contributor

No description provided.

@Neenu1995
Neenu1995 requested review from a team as code owners September 15, 2026 00:53

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

Copy link
Copy Markdown
Contributor

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 improves null-handling in BigQueryBaseResultSet by ensuring that getters for primitive types return appropriate default values when the underlying database value is null, and that object getters like getBlob and getClob safely return null. It also adds comprehensive integration tests to validate this behavior. The reviewer suggested an optimization in the new test helper method to retrieve ResultSetMetaData once outside the loop instead of repeatedly calling it inside the loop.

…ix-test-null-conversion

# Conflicts:
#	java-bigquery-jdbc/src/test/java/com/google/cloud/bigquery/jdbc/it/ITBigQueryJDBCTest.java
@Neenu1995
Neenu1995 enabled auto-merge (squash) September 15, 2026 02:04
@Neenu1995
Neenu1995 merged commit 33629c1 into main Sep 15, 2026
201 checks passed
@Neenu1995
Neenu1995 deleted the fix-test-null-conversion branch September 15, 2026 02:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants