jason810496 commented on code in PR #57741:
URL: https://github.com/apache/airflow/pull/57741#discussion_r2491087412
##########
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:
Review Comment:
> Could you please also enable dev mode when DEV_MODE env variable is set?
NP! I just added `should_enable_hot_reload` utils to check CLI args and both
`DEV_MODE` env.
> We can also consider splitting that to a different parameter or making
some kind of selective dev mode
Maybe the `--dev` flag at Breeze level should be provided with components
name (`all, api-server, scheduler, etc`), then toggle the `--dev` flag for
specified components? Or do we need different env variable name for each
component? (e.g. `<NAME>_DEV_MODE`)
I can refine detail on Breeze side in follow-up PR and this PR will only
change the airflow-core side.
--
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]
