Copilot commented on code in PR #86: URL: https://github.com/apache/pulsar-connectors/pull/86#discussion_r3559368329
########## hdfs3/src/test/java/org/apache/pulsar/io/hdfs3/sink/seq/HdfsTextSinkTest.java: ########## @@ -19,9 +19,11 @@ package org.apache.pulsar.io.hdfs3.sink.seq; import static org.mockito.Mockito.times; +import java.util.concurrent.TimeUnit; import static org.mockito.Mockito.verify; import static org.testng.Assert.assertNotNull; import org.apache.pulsar.io.hdfs3.sink.AbstractHdfsSinkTest; Review Comment: The new `TimeUnit` import splits the static-import block (`times` / `verify` / `assertNotNull`). Other tests in this module keep all static imports together, which is likely required by the project's import-order formatting/checkstyle rules. ########## hdfs3/src/test/java/org/apache/pulsar/io/hdfs3/sink/seq/HdfsSequentialSinkTest.java: ########## @@ -19,9 +19,11 @@ package org.apache.pulsar.io.hdfs3.sink.seq; import static org.mockito.Mockito.times; +import java.util.concurrent.TimeUnit; import static org.mockito.Mockito.verify; import static org.testng.Assert.assertNotNull; import org.apache.pulsar.io.hdfs3.sink.AbstractHdfsSinkTest; Review Comment: The added `TimeUnit` import is placed between static imports. This breaks the static-import grouping used elsewhere in this module and may fail automated formatting/import-order checks. -- 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]
