The helper can't be used under the scheduler's prohibit_commit: MetastoreBackend.get_variable's @provide_session opens the same scoped session and its commit trips the guard, so such callers must duplicate the backend walk.
Fix: optional keyword-only session=None, forwarded only to MetastoreBackend. Then the duplicated loop in serialization/definitions/dag.py can be deleted (see #68917).
The helper can't be used under the scheduler's
prohibit_commit:MetastoreBackend.get_variable's@provide_sessionopens the same scoped session and its commit trips the guard, so such callers must duplicate the backend walk.Fix: optional keyword-only
session=None, forwarded only toMetastoreBackend. Then the duplicated loop inserialization/definitions/dag.pycan be deleted (see #68917).