abhishekbhakat commented on code in PR #58111:
URL: https://github.com/apache/airflow/pull/58111#discussion_r2533546268
##########
airflow-core/src/airflow/serialization/serialized_objects.py:
##########
@@ -2111,7 +2116,10 @@ def _deserialize_field_value(cls, field_name: str,
value: Any) -> Any:
return None
elif field_name == "resources":
return Resources.from_dict(value) if value is not None else None
- elif field_name.endswith("_date"):
+ elif field_name in ("start_date", "end_date"):
+ # Skip deserialization for strings containing Jinja templates only
for top-level date fields
+ if isinstance(value, str) and cls._contains_jinja(value):
+ return value # Keep as-is for runtime rendering
Review Comment:
Ok cool. I'll refactor the PR to just raise broken dag. But I'll recommend
keep the try block on scheduler_job_runner as insurance.
--
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]
