You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "SamWheating (via GitHub)" <gi...@apache.org> on 2023/08/23 22:56:12 UTC

[GitHub] [airflow] SamWheating opened a new pull request, #33681: Set snowflake_conn_id on Snowflake Operators to avoid error

SamWheating opened a new pull request, #33681:
URL: https://github.com/apache/airflow/pull/33681

   closes: https://github.com/apache/airflow/issues/33679
   
   In https://github.com/apache/airflow/pull/30784 we added new template fields, but in the case of the snowflake operators, this attribute was never actually set during initialization (it was just passed to the parent class as `conn_id`). 
   
   As a result, these operators fail with the following exception:
   ```
   AttributeError: 'snowflake_conn_id' is configured as a template field but SnowflakeCheckOperator does not have this attribute.
   ```
   
   Anyways, this PR just adds the missing attributes.
   
   I replicated the original failure mode in Breeze, and then verified this fix.


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


[GitHub] [airflow] potiuk merged pull request #33681: Set snowflake_conn_id on Snowflake Operators to avoid error

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk merged PR #33681:
URL: https://github.com/apache/airflow/pull/33681


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