shibd commented on PR #1323: URL: https://github.com/apache/pulsar-client-go/pull/1323#issuecomment-2566372712
hi, @BewareMyPower 1. Even if we set the `connection` to null when receive cmd `ConsumerClosed`, we cannot completely guarantee that other API calls won't use the old connection to send to the old broker. Setting the connection to null and performing checks can help mitigate this issue to some extent. 2. If the `connection` is null when processing API requests (e.g., getLastMessageId), we should have an internal retry mechanism instead of throwing an exception to the client, as connection reconnection should be transparent to the caller. In this PR, I will throw an error when the connection is null. In a future PR, I will implement the retry logic. -- 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]
