RocMarshal commented on pull request #16417: URL: https://github.com/apache/flink/pull/16417#issuecomment-877924725
@LongWangXX Hi, I notice that the implements of `org.apache.flink.util.StringUtils#isNullOrWhitespaceOnly` is samilar to the implements of `org.apache.commons.lang3.StringUtils#isBlank`. IMO, We just need to keep consistent StringUtils calling `StringUtils#isBlank` without package prefix instead `org.apache.flink.util.StringUtils.isNullOrWhitespaceOnly(defaultTopic)` and keep the `StringUtils` reference located from the same package. Thanks for your attention. -- 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]
