kou commented on issue #36155: URL: https://github.com/apache/arrow/issues/36155#issuecomment-1654905888
We've added `CancelFlightInfo` action by #35500 but it accepts `FlightInfo` not `FlightDescriptor`. So we can't use it for cancelling a running query with `cancel_descriptor`. I think that we need to use `PollFlightInfo(cancel_descriptor)` instead of `CancelFlightInfo` action to cancel a running query but it may confuse users. Or we can remove `RetryInfo::cancel_descriptor` and use `CancelFlightInfo(RetryInfo::info)` instead. @lidavidm Do you have any opinion for this case? If you don't have any opinion, I'll use the removing `RetryInfo::cancel_descriptor` and using `CancelFlightInfo(RetryInfo::info)` approach. -- 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]
