hangc0276 commented on PR #3480: URL: https://github.com/apache/bookkeeper/pull/3480#issuecomment-1241496814
Nice catch! Please fix the check style. ``` Error: /home/runner/work/bookkeeper/bookkeeper/bookkeeper-server/src/main/java/org/apache/bookkeeper/proto/PacketProcessorBase.java:22:1: Import io.netty.channel.ChannelFuture appears after other imports that it should precede [ImportOrder] Error: /home/runner/work/bookkeeper/bookkeeper/bookkeeper-server/src/test/java/org/apache/bookkeeper/client/BookieRecoveryUseIOThreadTest.java:26:1: Extra separation in import group before 'java.io.IOException' [ImportOrder] Error: /home/runner/work/bookkeeper/bookkeeper/bookkeeper-server/src/test/java/org/apache/bookkeeper/client/BookieRecoveryUseIOThreadTest.java:26:1: Import java.io.IOException appears after other imports that it should precede [ImportOrder] Audit done. [INFO] There are 3 errors reported by Checkstyle 9.3 with buildtools/src/main/resources/bookkeeper/checkstyle.xml ruleset. Error: src/main/java/org/apache/bookkeeper/proto/PacketProcessorBase.java:[22,1] (imports) ImportOrder: Import io.netty.channel.ChannelFuture appears after other imports that it should precede Error: src/test/java/org/apache/bookkeeper/client/BookieRecoveryUseIOThreadTest.java:[26,1] (imports) ImportOrder: Extra separation in import group before 'java.io.IOException' Error: src/test/java/org/apache/bookkeeper/client/BookieRecoveryUseIOThreadTest.java:[26,1] (imports) ImportOrder: Import java.io.IOException appears after other imports that it should precede ``` -- 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]
