This is an automated email from the ASF dual-hosted git repository.
chia7712 pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/kafka.git
The following commit(s) were added to refs/heads/trunk by this push:
new 1260b6dbbea KAFKA-17806 remove this-escape suppress warnings in
AclCommand (#19256)
1260b6dbbea is described below
commit 1260b6dbbea4d7649d6e0273bd16e8302167b3dc
Author: PoAn Yang <[email protected]>
AuthorDate: Sat Mar 22 02:14:32 2025 +0800
KAFKA-17806 remove this-escape suppress warnings in AclCommand (#19256)
Change AclCommandOptions to final class, so we can remove
`@SuppressWarnings("this-escape")` on it.
Reviewers: Chia-Ping Tsai <[email protected]>
---
tools/src/main/java/org/apache/kafka/tools/AclCommand.java | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/tools/src/main/java/org/apache/kafka/tools/AclCommand.java
b/tools/src/main/java/org/apache/kafka/tools/AclCommand.java
index d54970ad042..b6c0175331d 100644
--- a/tools/src/main/java/org/apache/kafka/tools/AclCommand.java
+++ b/tools/src/main/java/org/apache/kafka/tools/AclCommand.java
@@ -404,7 +404,7 @@ public class AclCommand {
}
}
- public static class AclCommandOptions extends CommandDefaultOptions {
+ public static final class AclCommandOptions extends CommandDefaultOptions {
private final OptionSpec<String> bootstrapServerOpt;
private final OptionSpec<String> bootstrapControllerOpt;
@@ -430,7 +430,6 @@ public class AclCommand {
private final OptionSpecBuilder forceOpt;
private final OptionSpec<String> userPrincipalOpt;
- @SuppressWarnings("this-escape")
public AclCommandOptions(String[] args) {
super(args);
bootstrapServerOpt = parser.accepts("bootstrap-server", "A list of
host/port pairs to use for establishing the connection to the Kafka cluster." +
