lhotari opened a new issue #3093: URL: https://github.com/apache/bookkeeper/issues/3093
**BUG REPORT** ***Describe the bug*** The code in PendingReadOp doesn't handle failure cases. This code doesn't ever get executed: https://github.com/apache/bookkeeper/blob/017307bc67431a7616861ad09927c4e3327633d1/bookkeeper-server/src/main/java/org/apache/bookkeeper/client/PendingReadOp.java#L628-L645 The reason for this is that execution is guarded with the `complete` field: https://github.com/apache/bookkeeper/blob/017307bc67431a7616861ad09927c4e3327633d1/bookkeeper-server/src/main/java/org/apache/bookkeeper/client/PendingReadOp.java#L619-L622 In failure cases, this field is modified before calling the `submitCallback` method: https://github.com/apache/bookkeeper/blob/017307bc67431a7616861ad09927c4e3327633d1/bookkeeper-server/src/main/java/org/apache/bookkeeper/client/PendingReadOp.java#L168-L171 This is the reason why the code to release the entries is never executed. -- 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]
