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 2022/07/18 13:27:59 UTC

[GitHub] [airflow] eladkal commented on a diff in pull request #25121: Add "Optimizing" chapter to dynamic-dags section

eladkal commented on code in PR #25121:
URL: https://github.com/apache/airflow/pull/25121#discussion_r923372969


##########
docs/apache-airflow/howto/dynamic-dag-generation.rst:
##########
@@ -140,3 +140,20 @@ Each of them can run separately with related configuration
 
 .. warning::
   Using this practice, pay attention to "late binding" behaviour in Python loops. See `that GitHub discussion <https://github.com/apache/airflow/discussions/21278#discussioncomment-2103559>`_ for more details
+
+
+Optimizing DAG parsing in workers/Kubernetes Pods

Review Comment:
   Is it smart idea to mention k8s here?
   This is a general howto doc it also appears at the beginning of our documentation we can't expect users to be familiar with K8s here and the article contains alot of k8s references.
   Maybe the k8s related should be extracted to the k8s executor section with a reference note?



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