PERF-5725 Add support for snappy and zstd compression #47
Conversation
| <artifactId>core</artifactId> | ||
| <version>${project.version}</version> | ||
| </dependency> | ||
| <dependency> |
There was a problem hiding this comment.
Do we want to leave a link to the driver docs somewhere ?
There was a problem hiding this comment.
Edited patch description to include driver link. Is there any documentation worth updating?
There was a problem hiding this comment.
Could add the comment / link to the XML file too.
There was a problem hiding this comment.
added comments to both xmls
I don't think it's crucial as there is still small interest in the compressors. Also, if there will be ever decision to make one of the compressors default a new version of java driver needs to be released with required dependency to the chosen compressor. In such case it would be better to test new driver and merge that driver to the main ycsb repo.
(https://docs.devprod.prod.corp.mongodb.com/performance/workloads/YCSB_tests#parameterized-ycsb-tests)? |
Add explicit snappy and zstd compressors dependencies. See https://www.mongodb.com/docs/drivers/java/sync/current/fundamentals/connection/network-compression/ for more details. The compressors versions match mongodb driver optional dependencies.
Patch: here