Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

perf: use direct executor #1864

Merged
merged 3 commits into from
Dec 19, 2024
Merged

perf: use direct executor #1864

merged 3 commits into from
Dec 19, 2024

Conversation

olavloite
Copy link
Collaborator

Use a direct executor by default for JDBC connections, as these do not use the async API, and cancelling statements can be handled by directly in the JDBC driver. This reduces the overall number of threads that is created by the JDBC driver.

Use a direct executor by default for JDBC connections, as these do not
use the async API, and cancelling statements can be handled by directly
in the JDBC driver. This reduces the overall number of threads that is
created by the JDBC driver.
@olavloite olavloite requested a review from a team as a code owner December 14, 2024 12:31
@product-auto-label product-auto-label bot added size: m Pull request size is medium. api: spanner Issues related to the googleapis/java-spanner-jdbc API. labels Dec 14, 2024
@olavloite olavloite requested a review from rayudu3745 December 14, 2024 12:32
@olavloite olavloite merged commit 6d6d500 into main Dec 19, 2024
22 checks passed
@olavloite olavloite deleted the use-direct-executor branch December 19, 2024 14:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: spanner Issues related to the googleapis/java-spanner-jdbc API. size: m Pull request size is medium.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants