aloyszhang commented on PR #18293: URL: https://github.com/apache/pulsar/pull/18293#issuecomment-1305767966
> To my understanding, this PR canceled the retry action in putSchema, and add the retry action in putSchemaIfAbsent. But I notice the putSchema method will be called by the deleteSchema method, so this PR removes the retry functionality of deleteSchema method too. Will that be OK? As described in issue #18292, the retry function in `putSchema` is the root cause of creating lots of useless schema ledgers when the update zookeeper failed with `AlreadyExistsException` or `BadVersionException`. So, it should be removed. BTW, the retry function is only useful when there is a race condition updating the zookeeper, and the command of delete schema is invoked from the CLI or PulsarAdmin which is rare to have a high concurrency. So, little chance for delete the schema will fail. -- 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]
