[GitHub] [bookkeeper] AnonHxy opened a new pull request, #3935: Filter duplicated fence request for v2

["AnonHxy (via GitHub)" <gi...@apache.org>]

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

   ### Motivation
   
   The fence request should only process once. And the process could be a little long because of scaning rocksDB.  So the client could send many fence request for the same ledgerId because of client timeout.   The duplicated fence request will queue in the thread pool waiting to process.
   
   This patch will filter the duplicated fence request and put them into a `pendingMap`. The key of the map is `ReadRequest`. Once the first fence request complete,  all the other request in `pendingMap` with th same key will complete with the same result.
   
   ### Changes
   
   Add a `Map<BookieProtocol.ReadRequest, List<ReadEntryProcessor>> pendingFencing` to save the duplicated fence request


-- 
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] AnonHxy closed pull request #3935: Filter duplicated fence request for v2

["AnonHxy (via GitHub)" <gi...@apache.org>]

AnonHxy closed pull request #3935: Filter duplicated fence request for v2
URL: https://github.com/apache/bookkeeper/pull/3935


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