lhotari commented on issue #22242: URL: https://github.com/apache/pulsar/issues/22242#issuecomment-1988599913
> Maybe we should revert the [fix](https://github.com/apache/pulsar/pull/22023) at the old branch. The disabled/default value for `lookupTimeout` being 0 should be reasonable. However, these checks indeed affect the previous logic and should not be cherry-picked. I think that the behavior of `-1` and `0` for `lookupTimeout` should be consistent across master branch and older branches. If this isn't the case, it is a breaking change to Pulsar clients as the example of Flink Pulsar connector demonstrates. -- 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]
