paleolimbot commented on PR #13635: URL: https://github.com/apache/arrow/pull/13635#issuecomment-1249601594
This PR sets it for the entire session in R because filesystem objects persist between cancellable calls, and that's currently the main place where we can pass the StopToken (even though it's polled almost nowhere). Even if the stop token had nothing to do with the filesystem, there's no guarnatee that cancellable operations wouldn't be nested (e.g., flight-powered record batch reader as input into an exec plan). I would prefer to subclass the `StopToken` and use R's existing infrastructure for this, although I don't think this is possible with the current design of the stop source/token interface. -- 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]
