You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2022/07/16 02:38:28 UTC

[GitHub] [pulsar] shibd commented on pull request #16612: [fix][broker] BadVersionException when splitting bundles, delay 100ms and try again.

shibd commented on PR #16612:
URL: https://github.com/apache/pulsar/pull/16612#issuecomment-1186071764

   > LGTM, and I have a small question:
   > 
   > Is it a temporary solution? If it will affect the production env, should we consider discussing and completely solving this problem? Even we have to refactor this part of the code. I'm not sure about if it is deserved.
   
   I understand it like this:
   
   1. The production environment has always been without problems,  If the user concurrently calls split bundles failed, the user can retry. This is normal behavior.
   2. The processing of this PR is to increase the fault tolerance of concurrent split bundles, and minimize the chance of failure.


-- 
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