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

feat: collect int/float/boolean/date page-level statistics on write #1346

Merged
merged 20 commits into from
Oct 10, 2023

Conversation

rok
Copy link
Contributor

@rok rok commented Oct 3, 2023

This builds on top of #1100 and #1335 but only adds stats collection for int/float/boolean/date.

@rok rok force-pushed the stats-collection-subset branch 2 times, most recently from 8e7a83b to 9ca3ca8 Compare October 4, 2023 02:07
@rok rok requested a review from wjones127 October 4, 2023 02:07
Copy link
Contributor

@wjones127 wjones127 left a comment

Choose a reason for hiding this comment

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

@rok could you make a few more changes here? Then I think we are ready to move on.

rust/lance/src/io/writer/statistics.rs Outdated Show resolved Hide resolved
rust/lance/src/io/writer/statistics.rs Outdated Show resolved Hide resolved
rust/lance/src/io/writer/statistics.rs Outdated Show resolved Hide resolved
@rok rok force-pushed the stats-collection-subset branch 2 times, most recently from 979b234 to d5fc980 Compare October 6, 2023 21:32
@rok
Copy link
Contributor Author

rok commented Oct 6, 2023

Thanks for the review @wjones127 ! I think I've addressed everything.

@rok rok requested a review from wjones127 October 6, 2023 21:34
@rok
Copy link
Contributor Author

rok commented Oct 9, 2023

@wjones127 did you have any more input on this?

@rok rok merged commit 2b30e21 into lancedb:main Oct 10, 2023
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants