Skip to content
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.

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

storage: add a test for interactions between rejected commands and re… #32236

Merged
merged 1 commit into from
Nov 13, 2018

Conversation

andreimatei
Copy link
Contributor

…aders

Another test for #30792, this time testing the specific interaction
between readers and rejected Raft commands.

Release note: None

@andreimatei andreimatei requested a review from a team November 12, 2018 23:16
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@andreimatei
Copy link
Contributor Author

cc @tbg

Copy link
Member

@tbg tbg left a comment

Choose a reason for hiding this comment

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

Reviewed 4 of 4 files at r1.
Reviewable status: :shipit: complete! 0 of 0 LGTMs obtained

Copy link
Contributor

@bdarnell bdarnell left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewable status: :shipit: complete! 1 of 0 LGTMs obtained

@andreimatei
Copy link
Contributor Author

bors r+

@craig
Copy link
Contributor

craig bot commented Nov 13, 2018

Merge conflict

…aders

Another test for cockroachdb#30792, this time testing the specific interaction
between readers and rejected Raft commands.

Release note: None
@andreimatei
Copy link
Contributor Author

bors r+

craig bot pushed a commit that referenced this pull request Nov 13, 2018
32236: storage: add a test for interactions between rejected commands and re… r=andreimatei a=andreimatei

…aders

Another test for #30792, this time testing the specific interaction
between readers and rejected Raft commands.

Release note: None

Co-authored-by: Andrei Matei <[email protected]>
@craig
Copy link
Contributor

craig bot commented Nov 13, 2018

Build succeeded

@craig craig bot merged commit ef2549b into cockroachdb:master Nov 13, 2018
@andreimatei andreimatei deleted the jepsen-test branch November 14, 2018 06:31
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.

4 participants