hickey-ai opened a new pull request, #11465: URL: https://github.com/apache/rocketmq/pull/11465
### Which Issue(s) This PR Fixes Fixes #11462 ### Brief Description The most significant partial byte starts rendering at bitLength % 8, which includes one padding bit outside the logical range. Start at remainder minus one to retain the exact logical width and high-to-low order. Byte-aligned and empty rendering are preserved. This is independent of the constructor capacity change in #11459 and the earlier index checks in #10506. ### How Did You Test This Change? - Four new rendering methods cover logical width across eight non-byte-aligned lengths, nonzero padding, partial-byte order, whole-byte arrays and empty arrays. Three methods failed before the correction; all four passed after. - Independent Java 8/Maven 3.6.3 run: `mvn -B -pl filter -am -Dtest=BitsArrayTest,BitsArrayRenderingTest,BloomFilterTest,ExpressionTest,FilterSpiTest,ParserTest -Dsurefire.failIfNoSpecifiedTests=false -DfailIfNoTests=false test`. - All 113 tests in the seven filter test classes passed, 0 failures/errors/skips. Checkstyle and SpotBugs passed. Ancestor modules were compiled and checked; their full unit suites were not selected. - Entire published Git tree equals the independently tested local tree. This is a utility rendering contract fix, with no asserted live broker message-loss trigger. AI assistance: OpenAI Codex was used for investigation, implementation and tests; commits include tooling provenance. -- 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]
