arvindKandpal-ksolves commented on code in PR #4922:
URL: https://github.com/apache/cassandra/pull/4922#discussion_r3557323143
##########
src/java/org/apache/cassandra/tools/nodetool/Compact.java:
##########
@@ -23,16 +23,24 @@
import org.apache.cassandra.tools.NodeProbe;
import org.apache.cassandra.tools.nodetool.layout.CassandraUsage;
+import picocli.CommandLine.ArgGroup;
import picocli.CommandLine.Command;
import picocli.CommandLine.Option;
import picocli.CommandLine.Parameters;
+import static org.apache.cassandra.tools.nodetool.CommandUtils.concatArgs;
import static
org.apache.cassandra.tools.nodetool.CommandUtils.parseOptionalKeyspace;
import static
org.apache.cassandra.tools.nodetool.CommandUtils.parseOptionalTables;
import static org.apache.commons.lang3.StringUtils.EMPTY;
-// TODO CASSANDRA-20793 Types of input aguments shouldn't be mixed in the same
command. The keyspace, table and SSTable file arguments should have their own
commands.
-@Command(name = "compact", description = "Force a (major) compaction on one or
more tables or user-defined compaction on given SSTables")
+/**
+ * @deprecated See CASSANDRA-20793. Use {@code compact keyspace}, {@code
compact sstables},
+ * or {@code compact range} instead.
+ */
+@Deprecated(since = "7.0")
Review Comment:
I've set the deprecated version to 7.0 for now. Could you please confirm if
this is the correct version or if it should be 5.x?
--
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]
