-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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: Implements agg_list
for NullChunked
#15439
Conversation
@@ -183,6 +188,10 @@ impl SeriesTrait for NullChunked { | |||
NullChunked::new(self.name.clone(), self.len()).into_series() | |||
} | |||
|
|||
fn drop_nulls(&self) -> Series { | |||
NullChunked::new(self.name.clone(), 0).into_series() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not much, but it should be better than the default implementation.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, this is free.
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #15439 +/- ##
=======================================
Coverage 81.13% 81.13%
=======================================
Files 1362 1362
Lines 174820 174846 +26
Branches 2531 2531
=======================================
+ Hits 141836 141865 +29
+ Misses 32500 32497 -3
Partials 484 484 ☔ View full report in Codecov by Sentry. |
@@ -183,6 +188,10 @@ impl SeriesTrait for NullChunked { | |||
NullChunked::new(self.name.clone(), self.len()).into_series() | |||
} | |||
|
|||
fn drop_nulls(&self) -> Series { | |||
NullChunked::new(self.name.clone(), 0).into_series() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, this is free.
We must implements
agg_list
forNullChunked
, otherwise it will be dispatched to the default implementation which doesn't produce a list series.This fixes #15437.