chengw-netflix commented on code in PR #3752:
URL: https://github.com/apache/cassandra/pull/3752#discussion_r1909550095
##########
src/java/org/apache/cassandra/tools/nodetool/ListSnapshots.java:
##########
@@ -45,6 +45,21 @@ public class ListSnapshots extends NodeToolCmd
description = "Include ephememeral snapshots")
private boolean includeEphemeral = false;
+ @Option(title = "keyspace",
Review Comment:
yeah agreed. I didn't have the use case needing to filter by multiple
keyspace/tables.. it's just an additional feature nice to have and the ROI
seems to be reasonable. :-) but agreed let's leave it for future if it's
needed. It's non-blocking for this PR.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
