feat: add send and ack mutations Cloud Spanner Queues - #9200
Conversation
Update the system test instance to use ENTERPRISE edition for access to queue features.
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.
| }); | ||
| }); | ||
|
|
||
| describe('queueSend', () => { |
There was a problem hiding this comment.
currently we have added the test for happy path where we are providing all the optional parameter.
can we a test where we will not be passing these options payload and deliverTime and assert that mutations are not including those optional parameter
| }); | ||
| }); | ||
|
|
||
| describe('queueAck', () => { |
There was a problem hiding this comment.
similarly here, a test without options
Cloud Spanner Queue Mutation Support
Fixes #9199 🦕