CurtHagenlocher commented on code in PR #3217:
URL: https://github.com/apache/arrow-adbc/pull/3217#discussion_r2255016804
##########
csharp/src/Drivers/Databricks/DatabricksOperationStatusPoller.cs:
##########
@@ -69,13 +72,18 @@ private async Task PollOperationStatus(CancellationToken
cancellationToken)
var operationHandle = _statement.OperationHandle;
if (operationHandle == null) break;
Review Comment:
From a Power BI perspective, the most important use of cancellation is for
Direct Query because users can generate a lot of queries simply by clicking
around in a visual and in-progress queries will need to be cancelled if their
output is no longer needed. DQ output tends to be relatively small, so being
able to cancel in the middle of reading the output is arguably less important
than being able to cancel before the results start coming back.
--
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
