lhotari commented on PR #24366: URL: https://github.com/apache/pulsar/pull/24366#issuecomment-2988677917
> > @poorbarcode Please handle cherry-picking to branch-3.0 and branch-3.3 since there are merge conflicts. > > @poorbarcode branch-3.0 compilation fails in checkstyle. > > ``` > [INFO] There are 5 errors reported by Checkstyle 10.14.2 with ../buildtools/src/main/resources/pulsar/checkstyle.xml ruleset. > [ERROR] src/test/java/org/apache/pulsar/client/impl/RawReaderTest.java:[53,8] (imports) UnusedImports: Unused import: org.apache.pulsar.client.api.SubscriptionInitialPosition. > [ERROR] src/test/java/org/apache/pulsar/client/impl/RawReaderTest.java:[54,8] (imports) UnusedImports: Unused import: org.apache.pulsar.client.api.SubscriptionType. > [ERROR] src/test/java/org/apache/pulsar/client/impl/RawReaderTest.java:[55,8] (imports) UnusedImports: Unused import: org.apache.pulsar.client.impl.conf.ConsumerConfigurationData. > [ERROR] src/test/java/org/apache/pulsar/client/impl/RawReaderTest.java:[56,8] (imports) UnusedImports: Unused import: org.apache.pulsar.common.api.proto.BrokerEntryMetadata. > [ERROR] src/test/java/org/apache/pulsar/client/impl/RawReaderTest.java:[70,15] (imports) UnusedImports: Unused import: org.apache.pulsar.client.impl.RawReaderImpl.DEFAULT_RECEIVER_QUEUE_SIZE. > ``` > > After cherry-picking, please make sure that checkstyle passes and the project builds. A quick way to validate is to run this command: > > ``` > mvn -Pcore-modules,-main -T 1C clean install -DskipTests -Dspotbugs.skip=true -DnarPluginPhase=none > ``` @poorbarcode I fixed it in https://github.com/apache/pulsar/commit/b92af789 -- 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]
