Skip to content

Conversation

@nivykani
Copy link

Thanks for sending a pull request! Here are some tips if you're new here:

  • Ensure you have added or run the appropriate tests for your PR.
  • Be sure to keep the PR description updated to reflect all changes.
  • Write your PR title to summarize what this PR proposes.
  • If possible, provide a concise example to reproduce the issue for a faster review.
  • Read our contributor guidelines
  • If you're making a documentation change, see our guide to documentation contribution

Commit messages should follow the following format:

<One sentence description, usually Jira title or CHANGES.txt summary>

<Optional lengthier description (context on patch)>

patch by <Authors>; reviewed by <Reviewers> for CASSANDRA-#####

Co-authored-by: Name1 <email1>
Co-authored-by: Name2 <email2>

The Cassandra Jira

Copy link
Contributor

@jyothsnakonisa jyothsnakonisa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good overall, left a minor comment around the testing. Also, please correct JIRA information in the commit message and post the CI result in the JIRA with the Fuzz tests result this patch is trying to fix

@nivykani nivykani changed the title [CASSANDRA-146398996] Fix non-determinism of repair fuzz tests by passing randomizer to RetryStrategy [CASSANDRA-21087] Fix non-determinism of repair fuzz tests by passing randomizer to RetryStrategy Dec 19, 2025
@nivykani nivykani force-pushed the CASSANDRA-146398996 branch from 5be95f6 to 101645a Compare December 19, 2025 23:45
@nivykani nivykani force-pushed the CASSANDRA-146398996 branch from 101645a to 44200de Compare December 20, 2025 02:10
@nivykani nivykani force-pushed the CASSANDRA-146398996 branch from 44200de to 9bf9416 Compare December 20, 2025 02:13
Copy link
Contributor

@jyothsnakonisa jyothsnakonisa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants