Skip to content

chore(bqjdbc): helper tools#12973

Merged
logachev merged 5 commits intomainfrom
kirl/tools
May 1, 2026
Merged

chore(bqjdbc): helper tools#12973
logachev merged 5 commits intomainfrom
kirl/tools

Conversation

@logachev
Copy link
Copy Markdown
Contributor

Environments:

  • Terraform script to deploy GCE VM with configured Private Service Connect. Can be used to validate PSC properties in JDBC driver.

Tools:

  • Client: allows to run custom queries or metadata methods with various connection strings to debug the driver. Also makefile simplifies capturing JFR to analyze performance.
  • Perf: python script that leverages Client to run custom queries and compare perf between different driver versions. Allows to customize # of rows, columns, iterations & use rest or read api.

More details are in README docs for each tool

@logachev logachev requested review from a team as code owners April 30, 2026 22:19
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 adds a suite of tools for the BigQuery JDBC driver, including a Java-based client, performance benchmarking scripts, and Terraform configurations for Private Service Connect environments. Key feedback includes implementing try-with-resources for better resource management in the Java client, using System.nanoTime() for more accurate performance metrics, removing hardcoded usernames from Terraform outputs, and improving the portability of the Python performance script.

Comment thread java-bigquery/google-cloud-bigquery-jdbc/tools/perf/run_perf.py
@logachev logachev merged commit 856febf into main May 1, 2026
130 of 131 checks passed
@logachev logachev deleted the kirl/tools branch May 1, 2026 19:40
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.

3 participants