potiuk commented on pull request #16420:
URL: https://github.com/apache/airflow/pull/16420#issuecomment-860778954


   > Go discussion on whether this should be a user-defined parameter that 
could be over-written. The idea was for managed airflow providers (Astronomer, 
AWS), they could override this parameter for their installations of this repo. 
I guess they could override an non user-defined parameter as well. Any 
thoughts? I can talk to Ry over at Astronomer on this.
   
   I think it would be great to be able to override it without changing neither 
DAGs, nor provider code. but adding version of Airflow might be a good idea 
indeed.  @sfc-gh-madkins is this label somehow standardized on Snowflake side, 
or is it ok to add + version.version as it is in case of Google Providers?
   
   Ultimately maybe this will do?:
   
   ```
   os.enviroment.get('_SNOWFLAKE_AIRFLOW_LABEL', "AIRFLOW" + version.version) 
   ```
   


-- 
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:
[email protected]


Reply via email to