zhan7236 opened a new pull request, #2692: URL: https://github.com/apache/uniffle/pull/2692
### What changes were proposed in this pull request? Add unit tests to verify the `Roaring64NavigableMap` optimization introduced in PR #2687 for filtering duplicate blockIds from multiple replicas in `RssShuffleWriter#checkSentBlockCount`. ### Why are the changes needed? As suggested by the reviewer in PR #2687, tests should be added to cover the `Roaring64NavigableMap` optimization change. ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? New tests added: - `testRoaring64NavigableMapDeduplication`: Verifies correct deduplication of blockIds across multiple servers (replicas) - `testRoaring64NavigableMapWithLargeBlockIds`: Verifies behavior with large number of consecutive blockIds Test results: - Spark3 `RssShuffleWriterTest`: 9 tests passed (including 2 new tests) - Spark2 `RssShuffleWriterTest`: 5 tests passed (including 2 new tests) Related PR: #2687 -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
