Page MenuHomeFreeBSD

Add a section about Zstandard compression to the ZFS handbook
ClosedPublic

Authored by allanjude on Dec 22 2020, 3:27 AM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 4 2024, 10:44 AM
Unknown Object (File)
Jan 18 2024, 4:25 PM
Unknown Object (File)
Jan 2 2024, 5:32 PM
Unknown Object (File)
Dec 22 2023, 10:56 PM
Unknown Object (File)
Dec 21 2023, 2:25 AM
Unknown Object (File)
Dec 20 2023, 5:15 PM
Unknown Object (File)
Dec 14 2023, 12:01 AM
Unknown Object (File)
Nov 10 2023, 9:37 AM

Details

Summary

Sponsored by: The FreeBSD Foundation

Test Plan

Diff Detail

Repository
R9 FreeBSD doc repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 35589
Build 32489: arc lint + arc unit

Event Timeline

ygy added a subscriber: ygy.

Looks good to me on the doc side.

This revision is now accepted and ready to land.Dec 23 2020, 3:06 PM
emaste added inline comments.
en_US.ISO8859-1/books/handbook/zfs/chapter.xml
2948–2949

I wonder if it would be clear to say no advantage to lower compression levels to avoid the double negative. Or something like "but read many times, Zstd allows the use of high compression levels without a performance penalty."

Remove double negative as suggested by emaste

This revision now requires review to proceed.Dec 23 2020, 3:25 PM
This revision is now accepted and ready to land.Dec 23 2020, 3:28 PM

I can't vouch for the technical accuracy of this, not having the required knowledge of ZFS and OpenZFS, but other than that and a few nits, this looks fine.

en_US.ISO8859-1/books/handbook/zfs/chapter.xml
2946

After the initial introduction of both "Zstandard" and its acronym "Zstd", I'd use one consistently, not a mix of both.

2956

Clarification: here and in the following paragraph, is it ZFS or OpenZFS?

2981

That paragraph doesn't state how to change the compression from the default.

allanjude added inline comments.
en_US.ISO8859-1/books/handbook/zfs/chapter.xml
2946

Done

2956

They are fairly synonymous, I think we'll have to do an entire pass over the handbook at some point

2981

That paragraph doesn't state how to change the compression from the default.

That is covered in the properties chapter, not sure if it makes sense to repeat it here. It might.

allanjude marked an inline comment as done.

Cleanup suggested by Pauamma

This revision now requires review to proceed.Dec 23 2020, 4:48 PM

Cool, a nice addition to the handbook covering the newer ZFS features. Thanks for the writeup!

This revision is now accepted and ready to land.Dec 23 2020, 4:50 PM

Do I remember wrong, or does Zstd also offer the option to adaptively compress at as high a level as possible while maintaining bandwidth, or is that only for zfs send|receive? If so, should that also be mentioned?

en_US.ISO8859-1/books/handbook/zfs/chapter.xml
2940–2941

Should this be "is the default as of FreeBSD 13-CURRENT and will be in 13.0-RELEASE"?

2958

s/cache/memory/

en_US.ISO8859-1/books/handbook/zfs/chapter.xml
2958

s/cache/memory/

Or better yet:
<acronym>RAM</acronym> instead of the word cache or the word memory.

emaste added inline comments.
en_US.ISO8859-1/books/handbook/zfs/chapter.xml
2949

mmm, reading it again, maybe we should mention without a "read" performance penalty?

Do I remember wrong, or does Zstd also offer the option to adaptively compress at as high a level as possible while maintaining bandwidth, or is that only for zfs send|receive? If so, should that also be mentioned?

That is the command line zstd tool, ZFS doesn't have anything like this (yet)

en_US.ISO8859-1/books/handbook/zfs/chapter.xml
2949

agreed

allanjude marked an inline comment as done.

Update with more feedback from emaste and debdrup

This revision now requires review to proceed.Dec 24 2020, 3:06 PM

I think you've addressed all the feedback and had approvals from everyone attached ;)

This revision is now accepted and ready to land.Dec 24 2020, 6:51 PM