Skip to content

feat: teach RunEndArray NullCount and TrueCount#2007

Merged
danking merged 16 commits into
developfrom
dk/runend-stats
Jan 22, 2025
Merged

feat: teach RunEndArray NullCount and TrueCount#2007
danking merged 16 commits into
developfrom
dk/runend-stats

Conversation

@danking

@danking danking commented Jan 17, 2025

Copy link
Copy Markdown
Contributor

No description provided.

@danking danking requested a review from robert3005 January 17, 2025 22:17
@danking danking marked this pull request as ready for review January 17, 2025 22:18
Comment thread encodings/runend/src/array.rs Outdated

@robert3005 robert3005 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We should iterate validity imho

@danking

danking commented Jan 20, 2025

Copy link
Copy Markdown
Contributor Author

Benchmark added. Indeed, iterating validity is faster.

@danking danking requested a review from robert3005 January 20, 2025 17:48
Comment thread encodings/runend/src/array.rs Outdated
@danking danking requested a review from robert3005 January 21, 2025 10:46
Comment thread encodings/runend/src/array.rs Outdated
Comment thread encodings/runend/src/array.rs Outdated
@danking danking enabled auto-merge (squash) January 21, 2025 11:24
@danking danking requested a review from robert3005 January 21, 2025 12:09
Comment thread encodings/runend/src/statistics.rs Outdated
@danking danking requested a review from robert3005 January 22, 2025 14:34
@danking danking merged commit ee7abec into develop Jan 22, 2025
@danking danking deleted the dk/runend-stats branch January 22, 2025 15:45
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