Skip to content
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

Window Size for zstd-d #1

Closed
felixhandte opened this issue Mar 7, 2024 · 1 comment
Closed

Window Size for zstd-d #1

felixhandte opened this issue Mar 7, 2024 · 1 comment

Comments

@felixhandte
Copy link
Collaborator

The Compression Dictionary Transport spec introduces a new encoding scheme zstd-d. We may want to clarify the window limits for that scheme as well (either in this draft or in the CDT draft).

Probably the right answer is something like 8 MB or 2 * the dictionary size if one is in use, to allow delta-ing of larger resources.

@nidhijaju
Copy link
Owner

That's a good point!
I think in Chrome we ended up using max(dictionary size, 8 MB) if a dictionary is in use, or 8 MB otherwise.

I think that should be added to the Compression Dictionary Transport spec, since it's not published yet and that's where the Content-Encoding token for zstd-d is being added. I just filed an issue there.

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

No branches or pull requests

2 participants