Re: thread safety issue in the BK client, introduced in 4.9.0

2022-03-15 Thread Lari Hotari
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 see

Re: thread safety issue in the BK client, introduced in 4.9.0

2022-03-14 Thread Enrico Olivelli
Thank you Lari, great work I believe it is better to revert that change (I have sent a PR) We can then do performance tests and see if we need to do something or if actually performances are not affected so much Data corruption is a real problem Better performances is a "nice to have" Enrico I

thread safety issue in the BK client, introduced in 4.9.0

2022-03-14 Thread Lari Hotari
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 Threa