uranusjr commented on code in PR #74789:
URL: https://github.com/apache/airflow/pull/74789#discussion_r4343971216
##########
airflow-core/src/airflow/dag_processing/processor.py:
##########
@@ -578,6 +578,19 @@ def _execute_email_callbacks(dagbag: DagBag, request:
EmailRequest, log: Filteri
)
[email protected]
+class DagParseSource:
+ """
+ Bundle metadata snapshot shared by parses of the same refreshed source
tree.
+
+ ``version_data`` and its nested values must not be mutated after capture.
+ """
+
+ bundle_version: str | None = None
+ version_data: dict | None = None
Review Comment:
dict is mutable so the type is not really protected. It’s OK in practice if
this is intended to be Airflow implementation detail (is it?) but let’s change
the typing to `Mapping | None` and add `:meta private:` in the docstring.
--
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]
