hangc0276 commented on PR #18039: URL: https://github.com/apache/pulsar/pull/18039#issuecomment-1284771740
This PR has two issues - There are several skip tests, which contain a new bug introduced by this PR. https://github.com/apache/pulsar/pull/18125 ``` [INFO] Running org.apache.pulsar.broker.service.persistent.PersistentSubscriptionMessageDispatchStreamingDispatcherThrottlingTest Warning: Tests run: 133, Failures: 0, Errors: 0, Skipped: 133, Time elapsed: 0.074 s - in org.apache.pulsar.broker.service.persistent.PersistentSubscriptionMessageDispatchStreamingDispatcherThrottlingTest [INFO] Running org.apache.pulsar.broker.service.persistent.SimpleProducerConsumerTestStreamingDispatcherTest Warning: Tests run: 212, Failures: 0, Errors: 0, Skipped: 212, Time elapsed: 0.117 s - in org.apache.pulsar.broker.service.persistent.SimpleProducerConsumerTestStreamingDispatcherTest ``` - There are failed CIs. https://github.com/apache/pulsar/actions/runs/3247091530/jobs/5330460957 We'd better not merge it before all the CI tests passed. @Technoboy- @codelipenghui -- 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]
