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 2022/07/07 17:14:36 UTC

[GitHub] [arrow] dragosmg commented on a diff in pull request #13541: ARROW-15016: [R] `show_query()` for an `arrow_dplyr_query`

dragosmg commented on code in PR #13541:
URL: https://github.com/apache/arrow/pull/13541#discussion_r916107836


##########
r/src/arrowExports.cpp:
##########
@@ -4247,7 +4255,7 @@ BEGIN_CPP11
 END_CPP11
 }
 // recordbatch.cpp
-int RecordBatch__num_rows(const std::shared_ptr<arrow::RecordBatch>& x);
+r_vec_size RecordBatch__num_rows(const std::shared_ptr<arrow::RecordBatch>& x);

Review Comment:
   These changes (rows 4258 & 4853) were not added by me (arrowExports.cpp) is generated automatically. They are probably generated from code introduced sometime in the past.



-- 
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-unsubscribe@arrow.apache.org

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