[GitHub] [airflow] vandonr-amz commented on a diff in pull request #30495: preload airflow imports before dag parsing to save time

2023-04-14 Thread via GitHub
vandonr-amz commented on code in PR #30495: URL: https://github.com/apache/airflow/pull/30495#discussion_r1167001126 ## airflow/config_templates/config.yml: ## @@ -2344,6 +2344,16 @@ scheduler: version_added: 2.0.0 type: boolean default: "True" +parsing_

[GitHub] [airflow] vandonr-amz commented on a diff in pull request #30495: preload airflow imports before dag parsing to save time

2023-04-14 Thread via GitHub
vandonr-amz commented on code in PR #30495: URL: https://github.com/apache/airflow/pull/30495#discussion_r1167001126 ## airflow/config_templates/config.yml: ## @@ -2344,6 +2344,16 @@ scheduler: version_added: 2.0.0 type: boolean default: "True" +parsing_

[GitHub] [airflow] vandonr-amz commented on a diff in pull request #30495: preload airflow imports before dag parsing to save time

2023-04-13 Thread via GitHub
vandonr-amz commented on code in PR #30495: URL: https://github.com/apache/airflow/pull/30495#discussion_r1166056851 ## airflow/dag_processing/processor.py: ## @@ -187,6 +188,18 @@ def _handle_dag_file_processing(): def start(self) -> None: """Launch the process

[GitHub] [airflow] vandonr-amz commented on a diff in pull request #30495: preload airflow imports before dag parsing to save time

2023-04-13 Thread via GitHub
vandonr-amz commented on code in PR #30495: URL: https://github.com/apache/airflow/pull/30495#discussion_r1165866037 ## tests/dags/test_imports.nopy: ## @@ -0,0 +1,26 @@ +# using .nopy extension so that the file won't get reformatted +# because we want to assert that we can pars

[GitHub] [airflow] vandonr-amz commented on a diff in pull request #30495: preload airflow imports before dag parsing to save time

2023-04-13 Thread via GitHub
vandonr-amz commented on code in PR #30495: URL: https://github.com/apache/airflow/pull/30495#discussion_r1165857348 ## airflow/dag_processing/processor.py: ## @@ -187,6 +188,18 @@ def _handle_dag_file_processing(): def start(self) -> None: """Launch the process

[GitHub] [airflow] vandonr-amz commented on a diff in pull request #30495: preload airflow imports before dag parsing to save time

2023-04-13 Thread via GitHub
vandonr-amz commented on code in PR #30495: URL: https://github.com/apache/airflow/pull/30495#discussion_r1165856163 ## tests/dags/test_imports.nopy: ## @@ -0,0 +1,26 @@ +# using .nopy extension so that the file won't get reformatted +# because we want to assert that we can pars

[GitHub] [airflow] vandonr-amz commented on a diff in pull request #30495: preload airflow imports before dag parsing to save time

2023-04-12 Thread via GitHub
vandonr-amz commented on code in PR #30495: URL: https://github.com/apache/airflow/pull/30495#discussion_r1164316829 ## airflow/dag_processing/processor.py: ## @@ -187,6 +188,19 @@ def _handle_dag_file_processing(): def start(self) -> None: """Launch the process

[GitHub] [airflow] vandonr-amz commented on a diff in pull request #30495: preload airflow imports before dag parsing to save time

2023-04-11 Thread via GitHub
vandonr-amz commented on code in PR #30495: URL: https://github.com/apache/airflow/pull/30495#discussion_r1163098659 ## airflow/dag_processing/processor.py: ## @@ -187,6 +188,19 @@ def _handle_dag_file_processing(): def start(self) -> None: """Launch the process

[GitHub] [airflow] vandonr-amz commented on a diff in pull request #30495: preload airflow imports before dag parsing to save time

2023-04-10 Thread via GitHub
vandonr-amz commented on code in PR #30495: URL: https://github.com/apache/airflow/pull/30495#discussion_r1161882484 ## airflow/utils/file.py: ## @@ -371,3 +371,14 @@ def might_contain_dag_via_default_heuristic(file_path: str, zip_file: zipfile.Zi content = dag_fil

[GitHub] [airflow] vandonr-amz commented on a diff in pull request #30495: preload airflow imports before dag parsing to save time

2023-04-06 Thread via GitHub
vandonr-amz commented on code in PR #30495: URL: https://github.com/apache/airflow/pull/30495#discussion_r1160039420 ## airflow/utils/file.py: ## @@ -371,3 +371,14 @@ def might_contain_dag_via_default_heuristic(file_path: str, zip_file: zipfile.Zi content = dag_fil

[GitHub] [airflow] vandonr-amz commented on a diff in pull request #30495: preload airflow imports before dag parsing to save time

2023-04-06 Thread via GitHub
vandonr-amz commented on code in PR #30495: URL: https://github.com/apache/airflow/pull/30495#discussion_r1160029424 ## airflow/dag_processing/processor.py: ## @@ -187,6 +188,12 @@ def _handle_dag_file_processing(): def start(self) -> None: """Launch the process