Re: [PR] feat(csharp/src/Drivers/Databricks): Added support for user-configurable Fetch heartbeat interval param [arrow-adbc]

["CurtHagenlocher (via GitHub)" <gi...@apache.org>]

CurtHagenlocher commented on code in PR #3472:
URL: https://github.com/apache/arrow-adbc/pull/3472#discussion_r2376901131


##########
csharp/src/Drivers/Databricks/DatabricksParameters.cs:
##########
@@ -228,6 +228,22 @@ public class DatabricksParameters : SparkParameters
         /// Default value is false if not specified.
         /// </summary>
         public const string DriverConfigTakePrecedence = "adbc.databricks.driver_config_take_precedence";
+
+        /// <summary>
+        /// The interval in seconds for heartbeat polling during long-running operations.
+        /// This prevents queries from timing out by periodically checking operation status.
+        /// Default value is 60 seconds if not specified.
+        /// Must be a positive integer value.
+        /// </summary>
+        public const string FetchHeartbeatInterval = "adbc.databricks.fetch_heartbeat_interval";

Review Comment:
   It looks like `Databricks/readme.md` hasn't been kept up-to-date with the added configuration options. Rather than adding them as part of this PR, perhaps a separate PR could be submitted which adds all the outstanding values?



-- 
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-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org