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

chore: propagate RuntimeException from DelayedReadContext#3150

Merged
rahul2393 merged 1 commit intomainfrom
issue-3132
Jun 18, 2024
Merged

chore: propagate RuntimeException from DelayedReadContext#3150
rahul2393 merged 1 commit intomainfrom
issue-3132

Conversation

@olavloite
Copy link
Copy Markdown
Collaborator

Propagate the underlying cause of the ExecutionException as a RuntimeException directly, if this is already a RuntimeException. This keeps the behavior of multiplexed sessions and regular sessions equal, regardless whether the Delayed... route is taken for multiplexed sessions, or the direct route is used.

Fixes #3132

Propagate the underlying cause of the ExecutionException as
a RuntimeException directly, if this is already a RuntimeException.
This keeps the behavior of multiplexed sessions and regular
sessions equal, regardless whether the Delayed... route is
taken for multiplexed sessions, or the direct route is used.

Fixes #3132
@olavloite olavloite requested review from a team and rahul2393 June 10, 2024 07:58
@product-auto-label product-auto-label Bot added size: xs Pull request size is extra small. api: spanner Issues related to the googleapis/java-spanner API. labels Jun 10, 2024
@rahul2393 rahul2393 merged commit 71f56da into main Jun 18, 2024
@rahul2393 rahul2393 deleted the issue-3132 branch June 18, 2024 06:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: spanner Issues related to the googleapis/java-spanner API. size: xs Pull request size is extra small.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ITReadOnlyTxnTest is flaky with multiplexed sessions

2 participants