[GitHub] [airflow] potiuk commented on a change in pull request #22076: Add new options to DatabricksCopyIntoOperator

[GitBox <gi...@apache.org>]

potiuk commented on a change in pull request #22076:
URL: https://github.com/apache/airflow/pull/22076#discussion_r821589092



##########
File path: airflow/providers/databricks/operators/databricks_sql.py
##########
@@ -163,18 +163,23 @@ class DatabricksCopyIntoOperator(BaseOperator):
         or ``sql_endpoint_name`` must be specified.
     :param sql_endpoint_name: Optional name of Databricks SQL Endpoint.
         If not specified, ``http_path`` must be provided as described above.
-    :param files: optional list of files to import. Can't be specified together with ``pattern``.
+    :param files: optional list of files to import. Can't be specified together with ``pattern``. (templated)
     :param pattern: optional regex string to match file names to import.
         Can't be specified together with ``files``.
     :param expression_list: optional string that will be used in the ``SELECT`` expression.
+    :param credential: optional credential configuration for authentication against a specified location.

Review comment:
       It would be great to provide a link to the docs explaining what are possible credential/encryption options. And I think  example_dags should show a few examples (not only unit tests)




-- 
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: commits-unsubscribe@airflow.apache.org

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