bkmgit commented on pull request #11882: URL: https://github.com/apache/arrow/pull/11882#issuecomment-1003175374
> If functions were to be grouped and the `options` parameter used to select a particular function, then expressing function-specific `options` would be difficult. It is actually not too bad. [ScalarAggregationOptions](https://arrow.apache.org/docs/cpp/api/compute.html#_CPPv4N5arrow7compute22ScalarAggregateOptionsE) has two parameters, `skip_nulls` and `min_count`. Adding options with a default that preserves existing behavior makes extending kernels and updating language bindings easier. -- 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]
