Re: [PR] Add CLI hot-reload support via --dev flag [airflow]

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

amoghrajesh commented on code in PR #57741:
URL: https://github.com/apache/airflow/pull/57741#discussion_r2486398899


##########
airflow-core/src/airflow/cli/commands/dag_processor_command.py:
##########
@@ -52,6 +52,15 @@ def dag_processor(args):
     """Start Airflow Dag Processor Job."""
     job_runner = _create_dag_processor_job_runner(args)
 
+    if hasattr(args, "dev") and args.dev:
+        from airflow.cli.hot_reload import run_with_reloader
+
+        run_with_reloader(
+            lambda: run_job(job=job_runner.job, execute_callable=job_runner._execute),
+            process_name="dag-processor",
+        )
+        return

Review Comment:
   There are high chances or errors in code during development, how does the CLI handle it now?
   
   Let's say I make change to dag processor code where I try to import some wrong package, does it crash?



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