mustafasrepo commented on issue #9924: URL: https://github.com/apache/arrow-datafusion/issues/9924#issuecomment-2039021866
> FWIW @jayzhan211 has a really nice API suggestion here I think https://github.com/apache/arrow-datafusion/pull/9920/files#r1549905825 > > Specifically, add these two functions to the `AggregateUDFImpl` > > ```rust > /// Return true if the aggregate supports `IGNORE NULL`s > fn support_ignore_nulls() -> bool { false } > > /// Return true if the aggregate supports and `ORDER BY` to specify its ordering: > /// `SELECT first_value(x ORDER BY y) ... ` > fn support_ordering() -> bool { false } > ``` I think these APIs are very nice, clear. We can add these supports -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
