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