feat(spanner): add send and ack mutations Cloud Spanner Queues - #9200
feat(spanner): add send and ack mutations Cloud Spanner Queues#9200finnzzf wants to merge 6 commits into
Conversation
Update the system test instance to use ENTERPRISE edition for access to queue features.
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request introduces support for queue operations in Cloud Spanner by adding queueSend and queueAck methods to Transaction, MutationSet, and MutationGroup, along with corresponding options interfaces and mutation builders. It also adds unit and system tests for these new features. A review comment points out an issue in the system tests where the queue support flags are not set to false when the emulator is enabled, which would cause the tests to run and fail on the emulator.
|
presubmits are also failing, please fix that as well |
The lint errors do not seem to be related to my changes. They are coming from existing codes. |
finnzzf
changed the title
could you please run these two commands and push the changes?
|
|
/gemini review |
There was a problem hiding this comment.
Code Review
This pull request introduces support for queue send and ack mutations in Cloud Spanner transactions, including the necessary interfaces, builder functions, and methods across Transaction, MutationSet, and MutationGroup. It also adds corresponding unit and system tests. Feedback is provided to improve the system tests by cleaning up created queues to ensure test idempotency and to avoid using magic numbers for the instance edition.
Done |
|
/gcbrun |
2 participants
Cloud Spanner Queue Mutation Support
Fixes #9199 🦕