[GitHub] [pulsar] merlimat opened a new pull request, #16529: Fixed response when closing topic list watcher

[GitBox <gi...@apache.org>]

merlimat opened a new pull request, #16529:
URL: https://github.com/apache/pulsar/pull/16529

   ### Motivation
   
   I have seen some tests failing intermittently with producers/consumer stuck at close time, like: https://github.com/apache/pulsar/runs/7292496530?check_suite_focus=true 
   
   The error can be easily reproduced after a few attempts and it looks like that is related to the fact that the response to the `WatchTopicListClose` command is sent as a `CommandWatchTopicListSuccess` instead of a regular `CommandSuccess`. 
   
   The reason why it works sometime is that there is an exception when serializing the `CommandWatchTopicListSuccess` protobuf because the required "hash" field is missing. That triggers the connection to be closed. 
   
   Instead, the client side is expecting a regular `CommandSuccess` response.


-- 
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-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [pulsar] merlimat merged pull request #16529: Fixed response when closing topic list watcher

[GitBox <gi...@apache.org>]

merlimat merged PR #16529:
URL: https://github.com/apache/pulsar/pull/16529


-- 
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-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org