netudima commented on code in PR #2497:
URL: https://github.com/apache/cassandra/pull/2497#discussion_r2106310577
##########
src/java/org/apache/cassandra/tools/nodetool/GetSSTables.java:
##########
@@ -17,36 +17,48 @@
*/
package org.apache.cassandra.tools.nodetool;
-import static com.google.common.base.Preconditions.checkArgument;
-import io.airlift.airline.Arguments;
-import io.airlift.airline.Command;
-
import java.util.ArrayList;
import java.util.List;
import java.util.Map;
import java.util.Set;
-import io.airlift.airline.Option;
import org.apache.cassandra.tools.NodeProbe;
-import org.apache.cassandra.tools.NodeTool.NodeToolCmd;
+import org.apache.cassandra.tools.nodetool.layout.CassandraUsage;
+import picocli.CommandLine.Command;
+import picocli.CommandLine.Option;
+import picocli.CommandLine.Parameters;
+
+import static com.google.common.base.Preconditions.checkArgument;
+import static org.apache.cassandra.tools.nodetool.CommandUtils.concatArgs;
@Command(name = "getsstables", description = "Print the sstable filenames that
own the key")
-public class GetSSTables extends NodeToolCmd
+public class GetSSTables extends AbstractCommand
{
- @Option(title = "hex_format",
- name = {"-hf", "--hex-format"},
- description = "Specify the key in hexadecimal string format")
+ @Option(paramLabel = "hex_format",
+ names = { "-hf", "--hex-format" },
+ description = "Specify the key in hexadecimal string format")
private boolean hexFormat = false;
- @Option(name={"-l", "--show-levels"}, description="If the table is using
leveled compaction the level of each sstable will be included in the output
(Default: false)")
+ @Option(names = { "-l", "--show-levels" }, description = "If the table is
using leveled compaction the level of each sstable will be included in the
output (Default: false)")
private boolean showLevels = false;
- @Arguments(usage = "<keyspace> <cfname> <key>", description = "The
keyspace, the column family, and the key")
+ @CassandraUsage(usage = "<keyspace> <cfname> <key>", description = "The
keyspace, the column family, and the key")
private List<String> args = new ArrayList<>();
+ @Parameters(paramLabel = "keyspace", index = "0", arity = "0..1",
description = "The keyspace")
Review Comment:
should all the parameters be mandatory?
--
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: pr-unsubscr...@cassandra.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscr...@cassandra.apache.org
For additional commands, e-mail: pr-h...@cassandra.apache.org
