jaideeppyne opened a new pull request, #532: URL: https://github.com/apache/datasketches-cpp/pull/532
## Summary - deserialize the existing Java theta fixtures from their byte buffers - reserialize each sketch and require byte-for-byte parity - cover regular, compressed, and non-empty zero-retained sketches across 15 fixture cases This turns the existing semantic compatibility test into a regression harness for the binary discrepancies investigated in #460, including flags, seed hashes, and compressed preambles. ## Validation - `mvn -t /private/tmp/jdk25-toolchains.xml -P generate_java_files -Dtest=org.apache.datasketches.theta.ThetaSketchCrossLanguageTest test` in current `datasketches-java` - `cmake -S . -B build -DSERDE_COMPAT=true` - `cmake --build build -j2` - `ctest --test-dir build -R ^'theta_test$' --output-on-failure` (passes) - full CTest: all locally runnable targets pass; 10 other SerDe targets require their own Java fixture sets, which were not generated for this theta-only change -- 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]
