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

Reply via email to