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

Backwards incompatible change to String 'first'/'last' complex columns #13377

Closed
clintropolis opened this issue Nov 16, 2022 · 0 comments · Fixed by #13378
Closed

Backwards incompatible change to String 'first'/'last' complex columns #13377

clintropolis opened this issue Nov 16, 2022 · 0 comments · Fixed by #13378

Comments

@clintropolis
Copy link
Member

Affected Version

None yet 👍

Description

#12879 introduced compression for ingestion time string 'first' and 'last aggregators, which is cool, however it also changed it so that this is the only way these columns can be written, which is bad because it means that by default any cluster that upgrades can not roll back to an older version.

This code path needs modified to by default write out the existing segment format, and introduce an option to enable to compression. At a future date we could consider enabling the compression by default.

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

Successfully merging a pull request may close this issue.

1 participant