sijie commented on issue #4272: With multi-host HTTP client, stop after trying all the addresses URL: https://github.com/apache/pulsar/pull/4272#issuecomment-492424540 > This is not the first time something (perfectly avoidable) like this went in that broke master or production. I have explained at my first comment. The PR passed all the integration tests which didn't break master. The problem was that there are two concurrent merges, one set the default value to 5 minutes, one introduced multi-hosts feature. Such a problem can still happen if there are concurrent merges based on current CI pipeline, unless we add some sort of merge queues before merging.
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
