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

PARQUET-1566: [C++] Indicate if null count, distinct count are present in column statistics #8774

Closed
wants to merge 2 commits into from

Conversation

arw2019
Copy link
Contributor

@arw2019 arw2019 commented Nov 25, 2020

This PR adds flags and convenience methods to toggle between presence/absence of distinct_count and null_count column statistics when reading/writing Parquet files.

@github-actions
Copy link

@arw2019 arw2019 force-pushed the PARQUET-1566 branch 3 times, most recently from a4849fd to 81b7a5b Compare November 26, 2020 00:17
@pitrou pitrou changed the title PARQUET-1566: Indicate if null count, distinct count are present in column statistics PARQUET-1566: [C++] Indicate if null count, distinct count are present in column statistics Nov 26, 2020
@pitrou
Copy link
Member

pitrou commented Nov 26, 2020

Note: CI failure is unrelated.

Copy link
Member

@pitrou pitrou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1, thank you @arw2019

@pitrou pitrou closed this in 42066ec Nov 30, 2020
GeorgeAp pushed a commit to sirensolutions/arrow that referenced this pull request Jun 7, 2021
…t in column statistics

This PR adds flags and convenience methods to toggle between presence/absence of `distinct_count` and `null_count` column statistics when reading/writing Parquet files.

Closes apache#8774 from arw2019/PARQUET-1566

Authored-by: Andrew Wieteska <andrew.r.wieteska@gmail.com>
Signed-off-by: Antoine Pitrou <antoine@python.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants