freget commented on a change in pull request #19335:
URL: https://github.com/apache/airflow/pull/19335#discussion_r744172220
##########
File path: docs/apache-airflow-providers-databricks/connections/databricks.rst
##########
@@ -57,10 +58,19 @@ Password (optional)
Extra (optional)
Specify the extra parameter (as json dictionary) that can be used in the
Databricks connection.
- This parameter is necessary if using the *PAT* authentication method
(recommended):
+
+ Following parameter is necessary if using the *PAT* authentication method
(recommended):
* ``token``: Specify PAT to use.
+ Following parameters are necessary if using authentication with AAD token:
+
+ * ``azure_client_id``: ID of the Azure Service Principal
Review comment:
Why don't we put the client_id and client_secret into the user and
password fields of the connection? Ultimately, it's just a user and a password.
--
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]
