Skip to content

ankita/gocql #8275

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

ankita/gocql #8275

wants to merge 1 commit into from

Conversation

ankitaryan10
Copy link
Contributor

@ankitaryan10 ankitaryan10 commented Sep 3, 2025 •
edited
Loading

What changed?

Adding SpeculativeExecutionPolicy and SetIdempotent methods in Batch object.

  • SpeculativeExecutionPolicy is already defined on the batch object within gocql driver here.
  • SetIdempotent supports setting the idempotent field defined on the BatchEntry object here

Why?

Need to test SpeculativeExecutionPolicy for UpsertCurrentExecution flow within cds which is a batch operation.

How did you test it?

  • built
  • run locally and tested manually
  • covered by existing tests
  • added new unit test(s)
  • added new functional test(s)

@ankitaryan10 ankitaryan10 marked this pull request as ready for review September 3, 2025 03:24
@ankitaryan10 ankitaryan10 requested a review from a team as a code owner September 3, 2025 03:24
@ankitaryan10 ankitaryan10 requested a review from yycptt September 3, 2025 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yycptt yycptt Awaiting requested review from yycptt

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@ankitaryan10