Add compression using zstd #173
No reviewers
Labels
No labels
action
check-aws
action
discussion-needed
action
for-external-contributors
action
for-newcomers
action
more-info-needed
action
need-funding
action
triage-required
kind
correctness
kind
ideas
kind
improvement
kind
performance
kind
testing
kind
usability
kind
wrong-behavior
prio
critical
prio
low
scope
admin-api
scope
background-healing
scope
build
scope
documentation
scope
k8s
scope
layout
scope
metadata
scope
ops
scope
rpc
scope
s3-api
scope
security
scope
telemetry
No milestone
No project
No assignees
2 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: Deuxfleurs/garage#173
Loading…
Reference in a new issue
No description provided.
Delete branch "trinity-1686a/garage:compression"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
fix #27
at the moment, doing an integrity check on every node only verify integrity using zstd, it can't detect swapped blocks for instance. Should I make it so such a check is done?
I'll re-add some test to smoke-test with compressible data before marking this ready.Done!WIP: Add compression using zstdto Add compression using zstd@ -98,6 +100,18 @@ Never run a Garage cluster where that is not the case.**
Changing the `replication_mode` of a cluster might work (make sure to shut down all nodes
and changing it everywhere at the time), but is not officially supported.
### `compression_level`
To help people, I propose adding a bit more information here and maybe a link or two?
I found these resources:
d7e1736375/programs/zstd.1.md
9b97fdf74f/lib/dictBuilder/zdict.c (L768)
550410d05d/lib/zstd.h (L97)
38dfc4699e/programs/zstdcli.c (L774-L775)
The first link says:
The CLI explains the 3 different types of compression levels: standard, ultra and fast.
We could add something like that:
@ -30,6 +30,10 @@ dd if=/dev/urandom of=/tmp/garage.1.rnd bs=1k count=2 # No multipart, inline sto
dd if=/dev/urandom of=/tmp/garage.2.rnd bs=1M count=5 # No multipart but file will be chunked
dd if=/dev/urandom of=/tmp/garage.3.rnd bs=1M count=10 # by default, AWS starts using multipart at 8MB
dd if=/dev/urandom bs=1k count=2 | base64 -w0 > /tmp/garage.1.b64
We could a small comment to says that this has been added to test zstd compression?