ivankelly commented on a change in pull request #898: Bookies should prioritize
recovery reads/writes
URL: https://github.com/apache/bookkeeper/pull/898#discussion_r158288854
##########
File path:
bookkeeper-server/src/main/java/org/apache/bookkeeper/proto/BookieRequestProcessor.java
##########
@@ -342,7 +361,15 @@ private void processReadRequestV3(final
BookkeeperProtocol.Request r, final Chan
}
} else {
read = new ReadEntryProcessorV3(r, c, this, fenceThreadPool);
- if (null == readThreadPool) {
+
+ // If it's a recovery read, we want to make sure it gets submitted
right away,
+ // bypassing the eventual executor queue,
+ // to reduce the chances of ledger recovery exceptions.
+ boolean isRecoveryRead = r.getReadRequest().hasFlag()
Review comment:
Will do.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
