Skip to content

test(bigquery): Add tests to verify lossless max timestamp parsing#12892

Merged
jinseopkim0 merged 8 commits intomainfrom
bq-timestamp
Apr 22, 2026
Merged

test(bigquery): Add tests to verify lossless max timestamp parsing#12892
jinseopkim0 merged 8 commits intomainfrom
bq-timestamp

Conversation

@jinseopkim0
Copy link
Copy Markdown
Contributor

b/326260769

@jinseopkim0 jinseopkim0 marked this pull request as ready for review April 22, 2026 17:27
@jinseopkim0 jinseopkim0 requested review from a team as code owners April 22, 2026 17:27
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 unit and integration tests to verify the lossless handling of maximum timestamps when the useInt64Timestamps flag is enabled. The feedback focuses on improving the integration test by ensuring consistent method visibility, optimizing resource usage by reusing existing BigQuery options, and adding an assertion to verify that the query result set is not empty before validation.

jinseopkim0 and others added 3 commits April 22, 2026 13:30
…loud/bigquery/it/ITBigQueryTest.java

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
…loud/bigquery/it/ITBigQueryTest.java

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
…loud/bigquery/it/ITBigQueryTest.java

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
@jinseopkim0 jinseopkim0 enabled auto-merge (squash) April 22, 2026 17:31
@jinseopkim0 jinseopkim0 requested a review from lqiu96 April 22, 2026 18:12
@jinseopkim0 jinseopkim0 requested a review from lqiu96 April 22, 2026 18:48
@jinseopkim0 jinseopkim0 disabled auto-merge April 22, 2026 19:27
@jinseopkim0 jinseopkim0 merged commit 31213f8 into main Apr 22, 2026
122 checks passed
@jinseopkim0 jinseopkim0 deleted the bq-timestamp branch April 22, 2026 19:35
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