CurtHagenlocher commented on code in PR #3192:
URL: https://github.com/apache/arrow-adbc/pull/3192#discussion_r2240558548
##########
csharp/src/Drivers/Databricks/DatabricksParameters.cs:
##########
@@ -212,6 +212,12 @@ public class DatabricksParameters : SparkParameters
/// Default value is 0 (disabled) if not specified.
/// </summary>
public const string TokenRenewLimit =
"adbc.databricks.token_renew_limit";
+
+ /// <summary>
+ /// The client ID of the service principal when using workload
identity federation.
+ /// Default value is empty if not specified.
+ /// </summary>
+ public const string IdentityFederationClientId =
"adbc.databricks.identity_federation_client_id";
Review Comment:
Can this also be added to the documentation at
csharp\src\Drivers\Databricks\readme.md?
I haven't always been paying attention to that, so there may be a bit of a
gap now which could be addressed all at once with a separate PR.
--
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
