Pigsy-Monk opened a new pull request, #3154: URL: https://github.com/apache/fory/pull/3154
## Why? FastUtilSerializer provides optimized serialization for fastutil collections (such as `ShortOpenHashSet`, `ShortArrayList`, etc.), which are commonly used in high-performance applications. However, the existing implementation lacked comprehensive unit tests and performance benchmarks to validate its correctness and demonstrate the optimization benefits. This PR adds: 1. Comprehensive unit tests for `FastUtilSerializer` to ensure correctness 2. Performance tests comparing `FastUtilSerializer` with Fory's default serializer 3. Documentation of the optimization benefits, particularly for `ShortSet` serialization ## What does this PR do? ### 1. Added Test Dependencies - Added `fastutil` (version 8.5.6) as a test dependency in `fory-core/pom.xml` ### 2. Created Unit Tests (`FastUtilSerializerTest.java`) Comprehensive test coverage including: - `testShortOpenHashSet` - Tests serialization/deserialization of `ShortOpenHashSet` - `testShortArrayList` - Tests `ShortArrayList` serialization - `testIntArrayList`, `testLongArrayList`, `testByteArrayList` - Tests for various primitive array lists - `testObject2ObjectOpenHashMap` - Tests map serialization - `testRegisterAll` - Tests the `registerAll()` method - Empty collection tests - Large dataset tests ### 3. Created Performance Tests (`FastUtilShortSetPerformanceTest.java`) Performance comparison tests that demonstrate: - **Size comparison**: Compares serialized size between default serializer and FastUtilSerializer for different data sizes (10, 100, 1000, 10000 elements) - **Detailed optimization analysis**: Shows bytes saved and compression ratio - **Sparse data testing**: Tests with large gaps between values - **Dense data testing**: Tests with consecutive values - **Performance benchmark**: Measures serialization/deserialization time and speedup ### 4. Test Results The performance tests validate that: - FastUtilSerializer provides equal or better serialization efficiency compared to the default serializer - Serialization/deserialization correctness is maintained - The optimization benefits are measurable and documented ## Related issues -- 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]
