[GitHub] [airflow] eladkal commented on issue #33993: Confusing paragraph in Airflow Bestpractice documentaion (Airflow Variables)

["eladkal (via GitHub)" <gi...@apache.org>]

eladkal commented on issue #33993:
URL: https://github.com/apache/airflow/issues/33993#issuecomment-1702197853

   Thank you for trying to improve the docs!
   However I am not sure what is missing in the explnation?
   
   >I think some sentences should be added to explain why Variable.get() methods called from inside an Operator are a bad example.
   
   But it's OK to use `Variable.get()` in operator executed code like python callable of PythonOperator.
   Users should avoid using it in parts that are parsed by the scheduler.
   
   Could you check https://stackoverflow.com/a/71758614/14624409 if that clarify the issue we can work on modifying the docs to reflect it.


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