scott-routledge2 commented on code in PR #43698:
URL: https://github.com/apache/arrow/pull/43698#discussion_r2255384507


##########
cpp/src/arrow/dataset/scanner.h:
##########
@@ -107,6 +107,11 @@ struct ARROW_DS_EXPORT ScanOptions {
   /// Note: The IOContext executor will be ignored if use_threads is set to 
false
   io::IOContext io_context;
 
+  /// ExecContext for any CPU tasks
+  ///
+  /// Note: The ExecContext executor will be ignored if use_threads is set to 
false
+  compute::ExecContext exec_context;

Review Comment:
   I think it was an `ExecContext` originally because there are similar options 
in the compute module? Although I don't see any reason why it shouldn't be just 
`Executor*`, would be happy to make that change.



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

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

Reply via email to