Skip to content

fix(bigquery-jdbc): session context propagation when session is enabled - #14161

Open
Neenu1995 wants to merge 21 commits into
mainfrom
fix/temp-table-session-id
Open

fix(bigquery-jdbc): session context propagation when session is enabled#14161
Neenu1995 wants to merge 21 commits into
mainfrom
fix/temp-table-session-id

Conversation

@Neenu1995

@Neenu1995 Neenu1995 commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

b/543829068

@Neenu1995
Neenu1995 requested review from a team as code owners August 21, 2026 18:37

@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 implements session persistence across queries in the BigQuery JDBC driver by dynamically updating the connection's session information from query results and ensuring the session ID is correctly propagated in subsequent query configurations. It also adds corresponding unit and integration tests. The review feedback highlights a potential NullPointerException in BigQueryConnection if the transaction job is null, and suggests improving thread safety in BigQueryStatement by synchronizing access to connection properties using explicit locks and removing redundant null checks.

@Neenu1995 Neenu1995 changed the title fix(bigquery-jdbc): session contect propagation when session is enabled fix(bigquery-jdbc): session context propagation when session is enabled Aug 21, 2026
@keshavdandeva
keshavdandeva force-pushed the jdbc/remove-dry-run branch 2 times, most recently from 922d057 to 218cc5a Compare August 21, 2026 20:22
Base automatically changed from jdbc/remove-dry-run to main August 21, 2026 21:28
…gle-cloud-java into fix/temp-table-session-id
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