dongjinleekr commented on code in PR #16572:
URL: https://github.com/apache/kafka/pull/16572#discussion_r1677680044
##########
tools/src/main/java/org/apache/kafka/tools/ReplicaVerificationTool.java:
##########
@@ -112,6 +112,8 @@ public class ReplicaVerificationTool {
public static void main(String[] args) {
try {
+ LOG.warn("This tool is deprecated and may be removed in a future
major release.");
Review Comment:
@chia7712
I followed the consistency with `kafka.tools.MirrorMaker` and
`kafka.admin.FeatureCommand`.
`kafka.tools.MirrorMaker`:
```
bin/kafka-mirror-maker.sh --help
[2024-07-15 20:08:47,486] WARN This tool is deprecated and may be removed in
a future major release. (kafka.tools.MirrorMaker$)
This tool helps to continuously copy data between two Kafka clusters.
...
```
`kafka.admin.FeatureCommand`
```
bin/kafka-run-class.sh kafka.admin.FeatureCommand --help
WARNING: The 'kafka.tools' package is deprecated and will change to
'org.apache.kafka.tools' in the next major release.
usage: kafka-features [-h] [--command-config COMMAND_CONFIG]
(--bootstrap-server BOOTSTRAP_SERVER | --bootstrap-controller
BOOTSTRAP_CONTROLLER) {describe,upgrade,downgrade,disable} ...
This tool manages feature flags in Kafka.
...
```
If we really need the version, how about this? Is this okay?
> This tool is deprecated <ins>from 3.9.0</ins> and may be removed in a
future major release.
--
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]
