scott-routledge2 commented on code in PR #43698:
URL: https://github.com/apache/arrow/pull/43698#discussion_r2322206952
##########
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:
Unfortunately it looks like I don't have write access to the PR
title/description since I wasn't the one who opened it. Can you replace
ExecContext with Executor* in those places? I can also open a new PR and copy
my changes if that's easier.
--
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]
