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

Try reverting "compress on upload" #34536

Open
adeebshihadeh opened this issue Feb 6, 2025 · 1 comment
Open

Try reverting "compress on upload" #34536

adeebshihadeh opened this issue Feb 6, 2025 · 1 comment

Comments

@adeebshihadeh
Copy link
Contributor

#34528 broke compressed logs. It wasn't caught in CI since we use what's on disk in CI, which isn't what gets uploaded. We can add tests, but I think we should also investigate moving the compression back to loggerd.

IIRC we did that for two reason:

  1. CPU usage
  2. Getting as much as possible in the case of sudden power loss

#1 should be much better with zstd, and #2 is better defined now.

@adeebshihadeh
Copy link
Contributor Author

@deanlee want to try this out?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant