amoghrajesh commented on code in PR #67833:
URL: https://github.com/apache/airflow/pull/67833#discussion_r3334214142
##########
airflow-core/src/airflow/state/metastore.py:
##########
@@ -83,25 +83,25 @@ def _build_upsert_stmt(
return stmt
-class MetastoreStateBackend(BaseStateBackend):
+class MetastoreStateBackend(BaseStoreBackend):
Review Comment:
Good catch, renamed `MetastoreStateBackend` to `MetastoreStoreBackend` to be
consistent with the rest of the rename.
Left the module paths as is. Renaming them to `airflow.store.metastore`
would make the full config path
`airflow.store.metastore.MetastoreStoreBackend`, which feels like it introduces
more confusion than it resolves — "store" appears twice and the `airflow.state`
namespace has a reasonable meaning
--
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]
