datafusion icon indicating copy to clipboard operation
datafusion copied to clipboard

Support reading string view stats

Open XiangpengHao opened this issue 1 year ago • 3 comments

Which issue does this PR close?

Part of #11752.

Rationale for this change

Add implementation to support StringView in statistics. This should potentially allow StringView to run faster for parquet files that have good statistics.

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

XiangpengHao avatar Aug 07 '24 06:08 XiangpengHao

FWIW now that we implemented this for real in https://github.com/apache/arrow-rs/pull/6181 (thanks @Kev1n8 🙏 ) do we still need this code (once we have updated to get the changes upstream?)

alamb avatar Aug 19 '24 18:08 alamb

Marking as draft as I think this PR is no longer waiting on feedback. Please mark it as ready for review when it is ready for another look

alamb avatar Aug 19 '24 18:08 alamb

Thank you for your contribution. Unfortunately, this pull request is stale because it has been open 60 days with no activity. Please remove the stale label or comment or this will be closed in 7 days.

github-actions[bot] avatar Oct 19 '24 01:10 github-actions[bot]