Skip to content
This repository was archived by the owner on Mar 4, 2026. It is now read-only.

perf: increase default min sessions to 25#1167

Merged
olavloite merged 8 commits into
googleapis:masterfrom
olavloite:min-sessions-default-25
Jul 7, 2020
Merged

perf: increase default min sessions to 25#1167
olavloite merged 8 commits into
googleapis:masterfrom
olavloite:min-sessions-default-25

Conversation

@olavloite

Copy link
Copy Markdown
Contributor

Increase the default setting of SessionPoolOptions.min to 25.

Fixes #1160

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jul 6, 2020
@olavloite olavloite added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jul 6, 2020
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jul 6, 2020
@codecov

codecov Bot commented Jul 6, 2020

Copy link
Copy Markdown

Codecov Report

Merging #1167 into master will increase coverage by 0.10%.
The diff coverage is 94.87%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1167      +/-   ##
==========================================
+ Coverage   98.12%   98.23%   +0.10%     
==========================================
  Files          23       23              
  Lines       20996    21033      +37     
  Branches     1153     1156       +3     
==========================================
+ Hits        20603    20662      +59     
+ Misses        388      368      -20     
+ Partials        5        3       -2     
Impacted Files Coverage Δ
src/session-pool.ts 99.73% <94.87%> (+0.18%) ⬆️
src/common-grpc/service.ts 100.00% <0.00%> (+0.09%) ⬆️
src/v1/instance_admin_client.ts 99.26% <0.00%> (+0.10%) ⬆️
src/database.ts 99.92% <0.00%> (+0.11%) ⬆️
src/v1/database_admin_client.ts 99.56% <0.00%> (+0.12%) ⬆️
src/transaction-runner.ts 98.60% <0.00%> (+1.11%) ⬆️
src/backup.ts 99.61% <0.00%> (+1.14%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 66262e0...8fe50d8. Read the comment docs.

@olavloite olavloite requested a review from skuruppu July 6, 2020 19:25

@skuruppu skuruppu 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.

Thanks so much for this change.

Comment thread src/session-pool.ts Outdated
Comment thread src/session-pool.ts Outdated
Co-authored-by: skuruppu <skuruppu@google.com>
@olavloite olavloite requested review from a team July 7, 2020 13:08
@olavloite olavloite merged commit e4aba27 into googleapis:master Jul 7, 2020
@olavloite olavloite deleted the min-sessions-default-25 branch July 7, 2020 13:56
gcf-merge-on-green Bot pushed a commit that referenced this pull request Jul 7, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Allow a database client to be created with min > 0 sessions

4 participants