Wenzhi Feng
Wenzhi Feng
Could you help to reviw this PR? thanks! @eolivelli @dlg99 @hangc0276 @nicoloboschi @shoothzj @zymap @wenbingshen
> All the changes in this PR are specific to uncommon cases, occurring only when the entrylog file's index is missing. We have encountered cases that the ledger map in...
> Could you also shorten the method name? The test `testProducerDuplicationWithReceiptLostDedupEnabledAndUserControlSequenceIdMultiPartitioned` even has 90 characters. > > Currently, the checkstyle plugin does not work for tests somehow (see #23079). So...
> This problem seems to have been fixed on the master: #17795, but this PR has not been cherry-picked to 2.9.x I do not enable chunked messages. it seems like...
> Hello, I've also encountered this issue recently. May I ask if you have resolved it? We don't solve this problem, but avoid resending message in exception block.
Discuss link: https://lists.apache.org/thread/w965ygdgmn6yjpkp3lx23d3lo999tt88
Could you help to reviw this PR? @eolivelli @sijie @hangc0276 @zymap Thanks.
Could you help to reviw this PR? @eolivelli @dlg99 @hangc0276 @nicoloboschi @shoothzj @zymap @wenbingshen This bug could happen when the concurrent data structure shrink it's capacity.
I find that the same concurrency problem is reported in bookkeeper https://github.com/apache/bookkeeper/issues/4316, it is time to merge this fix into the bookkeeper. PTAL, thanks. @lhotari @hangc0276 @eolivelli @wenbingshen @zymap @shoothzj...
> Please fix the checkstyle > > ``` > > [INFO] There are 6 errors reported by Checkstyle 9.3 with buildtools/src/main/resources/bookkeeper/checkstyle.xml ruleset. > Error: src/test/java/org/apache/bookkeeper/util/collections/ConcurrentLongHashMapTest.java:[224,29] (whitespace) WhitespaceAfter: ',' is not...