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

fix: remove erroneous timeouts for batch_create_session calls #18

Merged
merged 4 commits into from Feb 19, 2020

Conversation

larkee
Copy link
Contributor

@larkee larkee commented Feb 18, 2020

Currently the timeout that is used for getting a session from the pool is mistakenly being used for the batch_create_sessions calls that are used to initially fill pools. If the timeout is short, the call does not have time to complete and the pools fails to initialize. Removing this timeout means the default timeout for BatchCreateSessions will be used.

@larkee larkee added the api: spanner Issues related to the googleapis/python-spanner API. label Feb 18, 2020
@larkee larkee requested a review from skuruppu February 18, 2020 04:23
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Feb 18, 2020
@larkee larkee merged commit 997a034 into googleapis:master Feb 19, 2020
@larkee larkee deleted the remove-erroneous-timeouts branch February 19, 2020 03:13
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/python-spanner API. cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants