You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/06/14 15:51:21 UTC

[GitHub] [airflow] kaxil edited a comment on pull request #16420: Added ability for Snowflake to attribute usage to Airflow by adding an application parameter

kaxil edited a comment on pull request #16420:
URL: https://github.com/apache/airflow/pull/16420#issuecomment-860793840


   > Are we sure we want it to be an operator parameter? I think it should be a connection configuration parameter or fixed constant. Now we force user to make changes in Python code.
   
   Good point, I like the idea of Connection better -- that way we can leverage environment Variables too


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

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