toddmeng-db commented on code in PR #3217:
URL: https://github.com/apache/arrow-adbc/pull/3217#discussion_r2249031292


##########
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:
   Actually, I think this depends a bit on how CancellationToken could be used 
by PBI, too
   @CurtHagenlocher will mashup ever trigger cancellationTokens passed into 
IArrowStreamReader.ReadNextBatchAsync? Do we need to ensure that the connection 
still remains usable for subsequent statements?



-- 
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]

Reply via email to