BewareMyPower opened a new pull request #13963: URL: https://github.com/apache/pulsar/pull/13963
Fixes #13955 ### Motivation [PIP-119](https://github.com/apache/pulsar/pull/13352) changes the default value of `subscriptionKeySharedUseConsistentHashing` with true, which makes the C++ test `KeySharedConsumerTest.testMultiTopics` flaky that not all messages can be received with 3 seconds receive timeout. It can be reproduce easily in my local env. I tried to use three Java consumers with `Key_Shared` subscription to consume the topic produced by C++ test `KeySharedConsumerTest.testMultiTopics`. Sometimes not all messages can be received as well. It looks like there is something wrong with the consistent hashing implementation of `Key_Shared` dispatcher. ### Modifications Since C++ UT is based on the standalone of config file `pulsar-client-cpp/test-conf/standalone-ssl.conf`, this PR disables the `subscriptionKeySharedUseConsistentHashing` in that configuration file. ### Verifying this change - [ ] Make sure that the change passes the CI checks. This change is a trivial rework / code cleanup without any test coverage. BTW, after applying the latest config, the test doesn't fail anymore in my local env. -- 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]
