amoghrajesh commented on code in PR #65759:
URL: https://github.com/apache/airflow/pull/65759#discussion_r3145893924
##########
airflow-core/src/airflow/config_templates/config.yml:
##########
@@ -2988,6 +2988,20 @@ dag_processor:
example: ~
default: "warning"
+state:
Review Comment:
Hmmm interesting take. Anything under `core` gives somewhat a sense of it
being a "core" configuration tied to airflow core. I personally do not like
`xcom_backend` placement in `core` and I think maintaining `state` as a
separate config can also support the future tasks where we will have more
configs come in like retention configurations. I think of state as a distinct
subsystem and prefer separate section but its arguable.
Any thoughts @ashb @kaxil @vikramkoka @jroachgolf84 @XD-DENG ?
--
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]
