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.

Sign up for GitHub

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

Jump to bottom

Configure zstd as default bucket encoding. #3594

Open
Dav1dde opened this issue May 14, 2024 · 1 comment
Open

Configure zstd as default bucket encoding. #3594

Dav1dde opened this issue May 14, 2024 · 1 comment
Labels
filler Requires little effort to resolve. Ready to be picked up anytime. maintenance Tasks with this label generally have a lower priority and due date

Comments

@Dav1dde
Copy link
Member

Dav1dde commented May 14, 2024

We currently support different encodings per namespace via global config, we should remove the global config options and use zstd by default for all namespaces.

  • Remove global config options from Sentry
  • Use zstd for all namespaces in Relay
@Dav1dde Dav1dde added the filler Requires little effort to resolve. Ready to be picked up anytime. label May 14, 2024
@jan-auer
Copy link
Member

I would suggest to set the defaults to what we have now configured in production:

  • zstd for distributions
  • base64 for sets. There have been cases we measured where zstd made things worse in sets.

@olksdr olksdr added the maintenance Tasks with this label generally have a lower priority and due date label May 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
filler Requires little effort to resolve. Ready to be picked up anytime. maintenance Tasks with this label generally have a lower priority and due date
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@Dav1dde @jan-auer @olksdr