unbridled-41 commented on PR #10992:
URL: https://github.com/apache/rocketmq/pull/10992#issuecomment-5547356168
Evidence chain re-verified on 2026-09-05 (JDK 21, Maven 3.8.7), anchored to
commits:
- **After fix** — fix commit 549aa9acb (the develop-merge on the branch is
from `update-branch`, no code change):
`mvn -pl broker test -Dtest=ConsumerOffsetManagerTest` → `Tests run: 6,
Failures: 0, Errors: 0, Skipped: 0`.
- **Before fix** — base commit e348efa66 with the same regression test file:
`mvn -pl broker test
-Dtest=ConsumerOffsetManagerTest#testConcurrentCommitOffsetDoesNotLoseQueues` →
`Tests run: 1, Failures: 1` — `java.lang.AssertionError: [offsets of
topic1] Expected size: 8 but was: 6 in: {0=1L, 1=1L, 3=1L, 4=1L, 5=1L, 7=1L}`
(ConsumerOffsetManagerTest.java:157): two of the eight concurrently committed
queues lost their offsets through the non-atomic `get`/`put` race.
- **CI**: all 10 checks green on this PR after re-triggering the workflows
via branch update (maven-compile linux/macos/windows, bazel, CodeQL, coverage,
license, misspell).
--
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]
