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


##########
airflow-core/src/airflow/state/metastore.py:
##########


Review Comment:
   Yeah, so we discussed on this on this PR itself and `airflow.state` is the 
infrastructure/backend layer, not the feature name — the feature is "task 
store" / "asset store". Renaming the package to airflow.store would make the 
default config path `airflow.store.metastore.MetastoreStoreBackend`, where 
"store" appears three times in one dotted path. Kafka Streams uses both words 
("State Store") precisely because the storage layer and the state management 
plumbing are distinct concepts so we dropped "State" from the user-facing names 
but keeping `airflow.state` for the backend package reflects that same 
separation. 
   
   



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

Reply via email to