The reported issue https://github.com/apache/bookkeeper/issues/3104 turned out to be a clear state handling issue in PendingReadOp class (including embedded classes), fix is https://github.com/apache/bookkeeper/pull/3110 .
-Lari On 2022/03/14 08:20:38 Lari Hotari wrote: > Hi all, > > There seems to be a severe thread safety issue in the BK client: > "Recycled LedgerEntryImpl instances are corrupted due to a thread safety > issue in BK client" > https://github.com/apache/bookkeeper/issues/3104 . > > I assume that we could simply revert PR "Read Submission should bypass OSE > Threads" https://github.com/apache/bookkeeper/pull/1792 . The commit for PR > was part of release 4.9.0 ( > https://github.com/apache/bookkeeper/commit/6b99ff73). > > This PR which broke the thread safety was made to address a performance > issue, "Read Submission should bypass OSE Threads" , > https://github.com/apache/bookkeeper/issues/1791. > > Since the proposal is to revert the performance optimization, the > alternative mitigation would be to implement > "Improve performance of OrderedExecutor by switching to a more performant > BlockingQueue implementation" > https://github.com/apache/bookkeeper/issues/3105 . > > Any feedback on this plan? > > There's already a PR https://github.com/apache/bookkeeper/pull/3106 to > revert PR "Read Submission should bypass OSE Threads" > https://github.com/apache/bookkeeper/pull/1792 . > Please review. > > BR, > > Lari >
