You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2021/04/13 08:30:59 UTC

[GitHub] [arrow] rok commented on a change in pull request #9841: ARROW-11070: [C++][Compute] Implement power kernel

rok commented on a change in pull request #9841:
URL: https://github.com/apache/arrow/pull/9841#discussion_r612241408



##########
File path: cpp/src/arrow/compute/kernels/scalar_arithmetic.cc
##########
@@ -359,6 +425,18 @@ const FunctionDoc div_checked_doc{
      "integer overflow is encountered."),
     {"dividend", "divisor"}};
 
+const FunctionDoc pow_doc{
+    "Raise arguments to power element-wise",
+    ("Integer to negative integer power returns an error. However, integer overflow\n"
+     "wraps around. Floating poing power follows std::pow() behaviour.\n"),

Review comment:
       We could add the cpp docs link the docstring?




-- 
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org