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

Pull in Upstream RLE and dictionary bug fixes #58

Closed
wilwade opened this issue Apr 27, 2022 · 0 comments · Fixed by #61
Closed

Pull in Upstream RLE and dictionary bug fixes #58

wilwade opened this issue Apr 27, 2022 · 0 comments · Fixed by #61
Assignees
@dopatraman
Labels
bug Something isn't working
Milestone
Release New Minor...

Comments

@wilwade
Copy link
Member

wilwade commented Apr 27, 2022

From: ZJONSSON#81

- RLE encoding and decoding does not work correctly if the dictionary has > 255 entries.
- A column chunk with a dictionary and data pages in both PLAIN and PLAIN_DICTIONARY decodes the PLAN pages incorrectly.
@wilwade wilwade added the bug Something isn't working label Apr 27, 2022
@wilwade wilwade moved this to 🧊 Icebox in DSNP and Frequency Project Apr 27, 2022
@wilwade wilwade moved this from 🧊 Icebox to 🪵 Backlog in DSNP and Frequency Project Apr 27, 2022
@wilwade wilwade added the planning Discuss & point in planning meeting label Apr 27, 2022
@dopatraman dopatraman self-assigned this Apr 29, 2022
@dopatraman dopatraman moved this from 🪵 Backlog to ⚙️ In Progress in DSNP and Frequency Project Apr 29, 2022
@wilwade wilwade removed the planning Discuss & point in planning meeting label May 9, 2022
@dopatraman dopatraman mentioned this issue May 9, 2022
Merged
Repository owner moved this from ⚙️ In Progress to ✅ Accepted in DSNP and Frequency Project May 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dopatraman dopatraman

Labels
bug Something isn't working
Projects
No open projects
DSNP and Frequency Project
Archived in project
Milestone
Release New Minor Version
Development

Successfully merging a pull request may close this issue.

Upstream bug fixes for RLE encoding LibertyDSNP/parquetjs
2 participants
@wilwade @dopatraman