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

metamorphic: fatal on nondeterministic single delete #4275

Closed
wants to merge 1 commit into from

Conversation

jbowens
Copy link
Collaborator

@jbowens jbowens commented Jan 21, 2025

Terminate and fail the metamorphic test if a nondeterministic single delete is observed. The generator is responsible for generating sequences of operations that ensure determinism across all operations, including single deletes.

Informs #4267.

Terminate and fail the metamorphic test if a nondeterministic single delete is
observed. The generator is responsible for generating sequences of operations
that ensure determinism across all operations, including single deletes.

Informs cockroachdb#4267.
@jbowens jbowens requested a review from a team as a code owner January 21, 2025 21:30
@jbowens jbowens requested a review from sumeerbhola January 21, 2025 21:30
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@RaduBerinde
Copy link
Member

I thought this can have false positives, as documented here:

pebble/event.go

Line 632 in 769e977

// This event can sometimes be a false positive because of delete-only

@jbowens
Copy link
Collaborator Author

jbowens commented Jan 21, 2025

Whoops. I re-read that comment but somehow missed it.

@jbowens jbowens closed this Jan 21, 2025
@jbowens jbowens deleted the nondeterm-singledel branch January 21, 2025 22:19
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