CASSANDRA-13391 Added --all to ClearSnapshot command #181
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Closes #123 Patch by dr3s and Jeff Jirsa; Reviewed by Stefan Podkowinski for CASSANDRA-13648
…he.java Closes #131 Patch by Ameya Ketkar; Reviewed by Jeff Jirsa for CASSANDRA-13732
patch by Simon Zhou; reviewed by Stefan Podkowinski for CASSANDRA-13387
Patch by Roman Pogribnyi; Reviewed by Jeff Jirsa for CASSANDRA-13689
Closes #130 Patch by Chris Lohfink; Reviewed by Jeremiah Jordan for CASSANDRA-13683
Patch by Chris Lohfink; reviewed by marcuse for CASSANDRA-11483
patch by Amitkumar Ghatwal and Michael Shuler; reviewed by Jeff Jirsa for CASSANDRA-13615
patch by Zhao Yang; reviewed by Stefan Podkowinski for CASSANDRA-13723
Patch by Blake Eggleston; Reviewed by Marcus Eriksson for CASSANDRA-13751
Patch by Corentin Chary; Reviewed by Chris Lohfink for CASSANDRA-13473
patch by Romain Hardouin; reviewed by Michael Shuler for CASSANDRA-12758
Patch by Blake Eggleston; Reviewed by Ariel Weisberg for CASSANDRA-13688
…start() Patch by marcuse; reviewed by Ariel Weisberg for CASSANDRA-13594
Patch by marcuse; reviewed by Ariel Weisberg for CASSANDRA-13664
…torRefTest Patch by Joel Knighton; reviewed by Marcus Eriksson for CASSANDRA-13594
Patch by marcuse; reviewed by Blake Eggleston for CASSANDRA-3200
Patch by Kevin Wern; Reviewed by Jeff Jirsa for CASSANDRA-13848
…tate pre-#14081 behavior)
Patch by Jay Zhuang; reviewed by Jeff Jirsa for CASSANDRA-14105
…ange itself) patch by Robert Stupp; reviewed by Andrés de la Peña for CASSANDRA-14109
Patch by marcuse; reviewed by Paulo Motta for CASSANDRA-13928
patch by Jay Zhuang; reviewed by Stefan Podkowinski for CASSANDRA-13077
patch by Jay Zhuang; reviewed by Stefan Podkowinski for CASSANDRA-14111
patch by Jay Zhuang; reviewed by Blake Eggleston for CASSANDRA-14106
patch by Jay Zhuang; reviewed by Stefan Podkowinski for CASSANDRA-14066
… command without a parameter will now throw an exception instead of deleting all the snapshots.
Contributor
Author
|
what the hell - it's the wrong branch. My mistake, clearly. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
20 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Added --all to ClearSnapshot command, so entering the command without a parameter will now throw an exception instead of deleting all the snapshots.