This repository was archived by the owner on Jan 26, 2026. It is now read-only.
Conversation
…mance tracking; add comprehensive String benchmark
…to remove a bunch of stuff
…ing deserialization a bit faster
A full list of enhancements can be found here - #3
…ome micro-optimizations added that were found along the way
* Full comprehensive comparison tests with a lot of other libraries + some micro-optimizations added that were found along the way * replace deprecated gradle methods with latest --------- Co-authored-by: expand3d <>
…with nested objects
…es in gradle file. Also fix permission issue
…Comparison tests iterations manually
added 23 commits
June 17, 2025 08:05
Adds Apache Thrift to the benchmark suite, including self-contained compiler download. Corrects Protobuf and FlatBuffers schemas and fixes bugs in the competitor classes to ensure a stable and robust benchmark environment. Includes refactored DataGenerator.
# Conflicts: # src/jmh/java/com/imprint/benchmark/ComparisonBenchmark.java # src/jmh/java/com/imprint/benchmark/FieldAccessBenchmark.java # src/jmh/java/com/imprint/benchmark/SerializationBenchmark.java # src/jmh/java/com/imprint/benchmark/serializers/ImprintSerializingBenchmark.java # src/main/java/com/imprint/core/ImprintFieldObjectMap.java # src/main/java/com/imprint/core/ImprintRecord.java # src/main/java/com/imprint/core/ImprintRecordBuilder.java # src/main/java/com/imprint/ops/ImprintOperations.java # src/main/java/com/imprint/types/TypeCode.java # src/main/java/com/imprint/types/TypeHandler.java # src/main/java/com/imprint/types/Value.java # src/test/java/com/imprint/IntegrationTest.java # src/test/java/com/imprint/ops/ImprintOperationsTest.java # src/test/java/com/imprint/profile/ProfilerTest.java
Collaborator
Author
There was a problem hiding this comment.
specific imprint benchmark to isolate just our own hot path
agavra
approved these changes
Jun 17, 2025
Contributor
agavra
left a comment
agavra
left a comment
There was a problem hiding this comment.
Skimmed the changes but it's a bit subtle and touches lots of code so it's hard to for me to review in detail. Overall LGTM.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
#20
Remove overhead caused by dynamic dispatch (important since we're competing with some aggressively optimized frameworks). The main work here was basically just getting rid of the
ValueandTypeHandlerclass and moving the logic into static/final classes that we can just invoke through case/switch branching instead of the interface/abstract strategy pattern (which was nice, but virtual overhead there seemed to be too much when going up against things like flatbuffers/protobuf/jackson). So it looks like a lot was changed but functionally, that's it - just making things static and breaking down the interfacePrevious best time for the serialization hotpath was about 5.1 microseconds and this update alone pulls it down to about 2.3 microseconds (maybe my computer just had a good day though because that seems like way too much of a gain, but JMH reports it). All tests passing and looking good so far.
Also I don't know why this keeps forcing me to rebase but I'll clean up the repo