fvaleri commented on code in PR #13562: URL: https://github.com/apache/kafka/pull/13562#discussion_r1285005842
########## tools/src/main/java/org/apache/kafka/tools/ToolsUtils.java: ########## @@ -99,4 +101,26 @@ public static void prettyPrintTable( printRow(columnLengths, headers, out); rows.forEach(row -> printRow(columnLengths, row, out)); } + + public static void validateBootstrapServer(String hostPort) throws IllegalArgumentException { + if (hostPort == null || hostPort.isEmpty()) { Review Comment: Since Java 11 we can use `isBlank()` that also ignores white spaces. Unfortunately, we still need to provide support for Java 8, where we can achieve the same by doing `hostPort.trim().isEmpty()`. -- 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: jira-unsubscr...@kafka.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org