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

Add pub api for checking column index is sorted #2848

Closed
Ted-Jiang opened this issue Oct 8, 2022 · 1 comment · Fixed by #2849
Closed

Add pub api for checking column index is sorted #2848

Ted-Jiang opened this issue Oct 8, 2022 · 1 comment · Fixed by #2849
Labels
parquet Changes to the parquet crate

Comments

@Ted-Jiang
Copy link
Member

Ted-Jiang commented Oct 8, 2022

I think parquet 23.0.0 will have properly exposed arrow definitions (added by @tustvold in #2626)

Though now that I see this, I see the proposal is probably to move the functions columns_sorted and check_is_ordered into the parquet crate which makes sense to me (and will likely get reviewed by some others with more parquet knowledge)

@Ted-Jiang can you file a ticket / PR to do so?

Originally posted by @alamb in apache/datafusion#3477 (comment)

@alamb
Copy link
Contributor

alamb commented Oct 14, 2022

label_issue.py automatically added labels {'parquet'} from #2849

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

Successfully merging a pull request may close this issue.

2 participants