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, the name of the current user _technically_ could change 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]
