Skip to content

fix(spanner): return sessions from pool in LIFO order - #9454

Merged
tseaver merged 2 commits into
googleapis:masterfrom
MaxxleLLC:spanner_lifo_sessions_pool
Nov 11, 2019
Merged

fix(spanner): return sessions from pool in LIFO order#9454
tseaver merged 2 commits into
googleapis:masterfrom
MaxxleLLC:spanner_lifo_sessions_pool

Conversation

@IlyaFaer

Copy link
Copy Markdown

Change sessions order from FIFO to LIFO. Done only in those pool classes, which don't have priorities.
Towards #9392

@IlyaFaerIlyaFaer added the api: spanner Issues related to the Spanner API. label Oct 11, 2019
@googlebotgooglebot added the cla: yes This human has signed the Contributor License Agreement. label Oct 11, 2019
Comment threadspanner/tests/unit/test_pool.py
Comment threadspanner/tests/unit/test_pool.py
@IlyaFaer
IlyaFaer marked this pull request as ready for review October 11, 2019 10:01
@tseavertseaver changed the title feat(spanner): lifo order in sessions poolfix(spanner): return sessions from pool in LIFO orderOct 15, 2019
@tswast
tswast requested review from crwilcox and removed request for tswastOctober 18, 2019 23:25
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: spannerIssues related to the Spanner API.cla: yesThis human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@IlyaFaer@larkee@tseaver@googlebot