poorbarcode commented on PR #20737: URL: https://github.com/apache/pulsar/pull/20737#issuecomment-1637484960
@BewareMyPower > I reverted the change of ConsumerImpl, testFirstSubscribeTimeout still succeeded. The description of the test in the section `Modifications`: ``` The test testFirstSubscribeTimeout only guarantees that subscribe timeout will not make an orphan consumer in the broker, not to produce the issue above ``` You should reproduce by the test which described: ``` The above process can be reproduced using the test testCnxInactiveWhenDoingSubscribe in the PR https://github.com/apache/pulsar/pull/20735. ``` -- 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]
