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

[TEST] Test dictionary encoding in ORC file for nested types with lots of rows. #8809

Closed
res-life opened this issue Jul 26, 2023 · 2 comments · Fixed by #8920
Closed

[TEST] Test dictionary encoding in ORC file for nested types with lots of rows. #8809

res-life opened this issue Jul 26, 2023 · 2 comments · Fixed by #8920
Assignees
@res-life
Labels
task Work required that improves the product but is not user facing test Only impacts tests

Comments

@res-life
Copy link
Collaborator

res-life commented Jul 26, 2023 •
edited
Loading

Describe the bug
As Bobby said, we have run into situations where having too many rows in nested columns causes problems with the dictionary.
Create this issue to follow up.

This is from the comment
Linked into the epic issue #8666
@res-life res-life added test Only impacts tests task Work required that improves the product but is not user facing labels Jul 26, 2023
This was referenced Jul 26, 2023
Open
Merged
@mythrocks
Copy link
Collaborator

Should the title read "Test Dictionary Encoding... "?

@res-life res-life changed the title [TEST] Test directory encoding in ORC file for nested types with lots of rows. [TEST] Test dictionary encoding in ORC file for nested types with lots of rows. Aug 1, 2023
@res-life res-life mentioned this issue Aug 3, 2023
Merged
@res-life
Copy link
Collaborator Author

Close it via #8920

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@res-life res-life

Labels
task Work required that improves the product but is not user facing test Only impacts tests
Projects
None yet
Milestone
No milestone
2 participants
@mythrocks @res-life