[C++] Clean up docs for log/trig functions, MakeUnaryArithmeticFunctionFloatingPoint

apache / arrow

Apache Arrow is the universal columnar format and multi-language toolbox for fast data interchange and in-memory analytics

https://arrow.apache.org/

Apache License 2.0

14.51k stars 3.53k forks source link

[C++] Clean up docs for log/trig functions, MakeUnaryArithmeticFunctionFloatingPoint #29035

Open asfimport opened 3 years ago

asfimport commented 3 years ago

The docs state these only work with float arguments, but they work with integers too.

MakeUnaryArithmeticFunctionFloatingPoint has a redundant check for float arguments that no longer applies: https://github.com/apache/arrow/blob/f7cac0e5866a08e946a14976ef854e8c42ac281b/cpp/src/arrow/compute/kernels/scalar_arithmetic.cc#L1266

Reporter: David Li / @lidavidm

_{Note: This issue was originally created as ARROW-13361. Please see the migration documentation for further details.}

asfimport commented 2 years ago

Todd Farmer / @toddfarmer: This issue was last updated over 90 days ago, which may be an indication it is no longer being actively worked. To better reflect the current state, the issue is being unassigned. Please feel free to re-take assignment of the issue if it is being actively worked, or if you plan to start that work soon.