nodece commented on PR #24638: URL: https://github.com/apache/pulsar/pull/24638#issuecomment-3204057549
> @nodece > > > Not a bug. Once the consumer checks the request timeout, it will send the close subscription request to the broker. > > Could you write a test for this case? @poorbarcode We only need to verify the logic in `handleSubscribe` and `handleCloseConsumer` in ServerCnx.java. The existing test `testSubscribeBookieTimeout` already covers this scenario and validates the changes introduced in this PR. -- 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: commits-unsubscr...@pulsar.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
