rok commented on code in PR #14434: URL: https://github.com/apache/arrow/pull/14434#discussion_r1025299474
########## docs/source/cpp/compute.rst: ########## @@ -450,6 +450,18 @@ Mixed time resolution temporal inputs will be cast to finest input resolution. +------------------+--------+------------------+----------------------+-------+ | divide_checked | Binary | Numeric/Temporal | Numeric/Temporal | \(1) | +------------------+--------+------------------+----------------------+-------+ +| exp | Unary | Numeric | Numeric | | Review Comment: Integer support for logarithms feels more important than here. But I suppose implicit casting will cover the gap if needed? Shall I remove the integer support then? -- 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: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
