eric-wang-1990 commented on code in PR #3589:
URL: https://github.com/apache/arrow-adbc/pull/3589#discussion_r2443605261


##########
csharp/src/Drivers/Databricks/DatabricksConnection.cs:
##########
@@ -84,6 +84,9 @@ internal class DatabricksConnection : SparkHttpConnection
         // Identity federation client ID for token exchange
         private string? _identityFederationClientId;
 
+        // Async execution poll interval configuration
+        private int _asyncExecPollIntervalMs = 
DatabricksConstants.DefaultAsyncExecPollIntervalMs;

Review Comment:
   Is there a follow up PR to map this parameter to the correct field? 
Basically I think you need this 
https://github.com/apache/arrow-adbc/blob/2d159d32cb04ff95b9750f3f23da4370a3488fbf/csharp/src/Drivers/Apache/Hive2/HiveServer2Statement.cs#L150.
   And after changing this you should verify the behavior by monitoring the 
Thrift calls to see whether it's sending poll request every 100ms.



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