paleolimbot commented on a change in pull request #12011:
URL: https://github.com/apache/arrow/pull/12011#discussion_r773852287
##########
File path: r/src/bridge.cpp
##########
@@ -21,13 +21,6 @@
#include <arrow/c/bridge.h>
-// [[arrow::export]]
Review comment:
Just a note to add this back...I'll take care of it when I do the R
ticket handling a potentially old Python version (either remove it and error or
use it for an old pyarrow).
##########
File path: r/tests/testthat/test-bridge.R
##########
@@ -25,16 +25,6 @@ test_that("Pointer wrapper accepts external pointers", {
delete_arrow_schema(ptr)
})
-test_that("Pointer wrapper accepts double-casted pointers", {
Review comment:
Same as above (I'll remove this in the R ticket if we're no longer using
it).
--
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]
