paleolimbot commented on code in PR #14582:
URL: https://github.com/apache/arrow/pull/14582#discussion_r1057317809
##########
r/src/compute-exec.cpp:
##########
@@ -30,6 +30,15 @@
#include <iostream>
#include <optional>
+// Can't find the header for this?
+namespace arrow {
+namespace io {
+namespace internal {
+arrow::internal::ThreadPool* GetIOThreadPool();
+}
+} // namespace io
+} // namespace arrow
Review Comment:
@pitrou this doesn't seem to be in any public header; however, its use is
required here for something that isn't R-specific (creating a source node from
a record batch reader). Should this be added to arrow/io/type_fwd.h instead of
declared here?
--
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]
