Skip to content

chore(Spanner): TransactionOptionsBuilder refactor (WIP)#8653

Closed
bshaffer wants to merge 50 commits intospanner-multiplexed-sessionsfrom
spanner-multiplexed-sessions-transaction-options-refactor
Closed

chore(Spanner): TransactionOptionsBuilder refactor (WIP)#8653
bshaffer wants to merge 50 commits intospanner-multiplexed-sessionsfrom
spanner-multiplexed-sessions-transaction-options-refactor

Conversation

@bshaffer
Copy link
Copy Markdown
Contributor

No description provided.

bshaffer added 30 commits May 12, 2025 19:41
  - explicitly selecting valid fields
  - better names for variables
  - documenting missing options in phpdoc
  - removing pass-by-reference
 - per-component phpstan
 - upgrade spanner to phpstan level 3
 - improved typing
 - address review comments
@bshaffer bshaffer changed the title TransactionOptionsBuilder WIP chore: [Spanner] TransactionOptionsBuilder refactor (WIP) Oct 10, 2025
@bshaffer bshaffer added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Oct 10, 2025
@bshaffer bshaffer mentioned this pull request Oct 10, 2025
3 tasks
@bshaffer bshaffer force-pushed the spanner-multiplexed-sessions-transaction-options-refactor branch from 8e6bcf9 to 0b1ed7c Compare October 10, 2025 18:00
@bshaffer bshaffer added v2 api: spanner Issues related to the Spanner API. labels Oct 10, 2025
@bshaffer bshaffer force-pushed the spanner-multiplexed-sessions branch 3 times, most recently from a7a0397 to 60b6801 Compare October 15, 2025 18:51
@bshaffer bshaffer changed the title chore: [Spanner] TransactionOptionsBuilder refactor (WIP) chore(Spanner): TransactionOptionsBuilder refactor (WIP) Oct 22, 2025
@bshaffer bshaffer closed this Oct 22, 2025
@bshaffer bshaffer deleted the spanner-multiplexed-sessions-transaction-options-refactor branch October 22, 2025 18:01
@bshaffer
Copy link
Copy Markdown
Contributor Author

replaced by #8681

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 Spanner API. do not merge Indicates a pull request not ready for merge, due to either quality or timing. v2

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant