Skip to content

doc: include reference to lz4 in akka.serialization.jackson.compression #32757

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 30, 2025

Conversation

leviramsey
Copy link
Contributor

If looking at the reference config, the availability of lz4 for gzip isn't clear since the only reference is when jackson-json overrides compression. The paradox docs do mention this so it's not entirely undocumented.

Semi-related: Jackson's usage of CBOR entails writing field names as values, so for objects that have fieldNamesWhichAreParticularlyLong (not uncommon in Java), CBOR is also rather verbose (and fairly compressible once you start having collections of objects with such field names), so is it worth enabling compression by default in CBOR?

Copy link
Contributor

@patriknw patriknw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, I think we can leave the defaults as is. It's documented

For the jackson-cbor and custom bindings other than jackson-json compression is by default disabled, but can be enabled in the same way as the configuration shown above but replacing jackson-json with the binding name (for example jackson-cbor).

@johanandren johanandren reopened this Jun 30, 2025
@johanandren johanandren merged commit 0ceabae into akka:main Jun 30, 2025
9 of 13 checks passed
@leviramsey leviramsey deleted the lz4-doc branch June 30, 2025 10:51
@johanandren johanandren added this to the 2.10.7 milestone Jul 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants