eric-wang-1990 commented on code in PR #3589:
URL: https://github.com/apache/arrow-adbc/pull/3589#discussion_r2448317873
##########
csharp/src/Drivers/Databricks/DatabricksParameters.cs:
##########
@@ -254,6 +255,15 @@ public class DatabricksParameters : SparkParameters
/// Must be a positive integer value.
/// </summary>
public const string OperationStatusRequestTimeout =
"adbc.databricks.operation_status_request_timeout";
+
+ /// <summary>
+ /// The interval in milliseconds between polls while waiting for query
execution to complete.
+ /// This controls how frequently the driver checks if an executing
query has finished.
+ /// Uses the Apache base parameter (adbc.apache.statement.polltime_ms)
but with Databricks-specific default.
+ /// Default value is 100 milliseconds if not specified.
+ /// Must be a positive integer value.
+ /// </summary>
+ public const string AsyncExecPollInterval =
ApacheParameters.PollTimeMilliseconds;
Review Comment:
Why do we need another variable? Can we just use
ApacheParameters.PollTimeMilliseconds?
--
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]
