You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "eladkal (via GitHub)" <gi...@apache.org> on 2023/03/10 17:02:42 UTC

[GitHub] [airflow] eladkal commented on a diff in pull request #30020: Update documentation for snowflake provider 4.0 breaking change

eladkal commented on code in PR #30020:
URL: https://github.com/apache/airflow/pull/30020#discussion_r1132629345


##########
airflow/providers/snowflake/operators/snowflake.py:
##########
@@ -42,7 +42,7 @@ class SnowflakeOperator(SQLExecuteQueryOperator):
         a list of str (sql statements), or a reference to a template file.
         Template references are recognized by str ending in '.sql'
     :param autocommit: if True, each command is automatically committed.
-        (default value: True)
+        (default value: False)

Review Comment:
   I suggest to remove it all together (this is what we usually do)
   The default value and the description is taken from the parent class



-- 
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