kaxil commented on code in PR #65759:
URL: https://github.com/apache/airflow/pull/65759#discussion_r3153690832
##########
airflow-core/src/airflow/config_templates/config.yml:
##########
@@ -2997,6 +2997,20 @@ dag_processor:
example: ~
default: "warning"
+state:
Review Comment:
Re-flagging the section-name concern from the prior round. `state` is
heavily overloaded in Airflow (`TI.state`, `DR.state`, `TaskInstanceState`,
`state_color_*`, etc.), so users will read `[state]` and reasonably expect it
to be about TI/DR state behaviour, not a key/value blob backend.
`[state_store]` parallels `[secrets]` and reads cleanly; `[task_state]` also
works. Worth picking now since AIP-103 will likely add per-backend keys
(retention, GC) in the next PRs and renaming a config section after release is
painful.
--
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]
