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

Posted to commits@bookkeeper.apache.org by GitBox <gi...@apache.org> on 2022/08/19 22:17:22 UTC

[GitHub] [bookkeeper] karanmehta93 opened a new pull request, #3452: WriteLacResponse should be processed in the same thread as other requ…

karanmehta93 opened a new pull request, #3452:
URL: https://github.com/apache/bookkeeper/pull/3452

   …ests for same ledgerId
   
   In PCBC code, the WriteLacCompletion object uses LAC as the key for selecting the thread in which the response
   will be processed. However, according to bookkeeper code guarantees, the thread should be decided based on hash of ledgerId.


-- 
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@bookkeeper.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [bookkeeper] eolivelli commented on pull request #3452: WriteLacResponse should be processed in the same thread as other requ…

Posted by GitBox <gi...@apache.org>.

eolivelli commented on PR #3452:
URL: https://github.com/apache/bookkeeper/pull/3452#issuecomment-1221480443

   @merlimat @RaulGracia @rakeshadr @nicoloboschi FYI


-- 
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@bookkeeper.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [bookkeeper] StevenLuMT commented on pull request #3452: WriteLacResponse should be processed in the same thread as other requ…

Posted by GitBox <gi...@apache.org>.

StevenLuMT commented on PR #3452:
URL: https://github.com/apache/bookkeeper/pull/3452#issuecomment-1221233293

   the old workflow has error,the bug  is fixed in #3451 
   please rerun the new workflow: 
   -  reopen the pr
   - or add some the new comments
   


-- 
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@bookkeeper.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [bookkeeper] eolivelli merged pull request #3452: WriteLacResponse should be processed in the same thread as other requ…

Posted by GitBox <gi...@apache.org>.

eolivelli merged PR #3452:
URL: https://github.com/apache/bookkeeper/pull/3452


-- 
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@bookkeeper.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org