Skip to content

Conversation

@olavloite
Copy link
Collaborator

Sessions were added to a set of checked out sessions when one was checked out from
the session pool. When the session was released back to the pool, the session would
normally be removed from the set of checked out sessions. The latter would not always
happen if the application that checked out the session did not use the session for
any reads or writes.

Sessions were added to a set of checked out sessions when one was checked out from
the session pool. When the session was released back to the pool, the session would
normally be removed from the set of checked out sessions. The latter would not always
happen if the application that checked out the session did not use the session for
any reads or writes.

chore: fix maven version to 3.8.1

chore: fix maven version for windows
@olavloite olavloite requested review from a team as code owners September 24, 2021 08:34
@generated-files-bot
Copy link

Warning: This pull request is touching the following templated files:

  • .github/workflows/ci.yaml

@product-auto-label product-auto-label bot added the api: spanner Issues related to the googleapis/java-spanner API. label Sep 24, 2021
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Sep 24, 2021
@thiagotnunes thiagotnunes changed the title fix: sessions were not always removed from checkedOutSessions fix(5.2.x): sessions were not always removed from checkedOutSessions Sep 24, 2021
@thiagotnunes thiagotnunes added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 29, 2021
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 29, 2021
@thiagotnunes thiagotnunes added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 29, 2021
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 29, 2021
@codecov
Copy link

codecov bot commented Sep 29, 2021

Codecov Report

❗ No coverage uploaded for pull request base (5.2.x@b7a911a). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##             5.2.x    #1447   +/-   ##
========================================
  Coverage         ?   85.29%           
  Complexity       ?     2662           
========================================
  Files            ?      146           
  Lines            ?    14409           
  Branches         ?     1396           
========================================
  Hits             ?    12290           
  Misses           ?     1535           
  Partials         ?      584           

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 b7a911a...c3c9b92. Read the comment docs.

@thiagotnunes thiagotnunes merged commit a434730 into 5.2.x Sep 29, 2021
@thiagotnunes thiagotnunes deleted the remove-from-checked-out-5.2.x branch September 29, 2021 04:17
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 API. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants