ashb commented on a change in pull request #20722:
URL: https://github.com/apache/airflow/pull/20722#discussion_r780172962



##########
File path: airflow/utils/platform.py
##########
@@ -63,6 +65,7 @@ def get_airflow_git_version():
     return git_version
 
 
+@cache

Review comment:
       I added this as there is 0 need to call this 20k times when creating a 
dag -- it can't change at runtime.
   
   (Okay, it _technically_ could change the name at runtime, but that is very 
much an edge case, and not to mention that the value created in the scheduler 
doesn't matter, it will be re-overwritten when the Task executes, and that is 
the interesting value anyway




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

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to