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/06/14 09:08:35 UTC

[GitHub] [arrow] thisisnic commented on pull request #10519: ARROW-12867: [R] Bindings for abs()

thisisnic commented on pull request #10519:
URL: https://github.com/apache/arrow/pull/10519#issuecomment-860526967


   Nice, thanks for submitting this PR!  There is a test failure caused by the calls to `abs()` on lines 146 and 149 in `test-dplyr-arrange.R` - the test they're part of previously was expected to result in a warning message as the function `abs` was not supported, but now you've added this functionality, no warning is issued.  


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