kaxil commented on code in PR #63783:
URL: https://github.com/apache/airflow/pull/63783#discussion_r2956611925
##########
airflow-core/src/airflow/partition_mappers/temporal.py:
##########
@@ -63,7 +63,7 @@ def deserialize(cls, data: dict[str, Any]) -> PartitionMapper:
)
-class HourlyMapper(_BaseTemporalMapper):
+class ToHourlyMapper(_BaseTemporalMapper):
Review Comment:
~~DAGs serialized before this rename store the fully-qualified path in the
metadata DB. After this rename, `decode_partition_mapper` calls
`import_string()` with that stored path and it will fail with an
`ImportError`.~~
**Edit:** Disregard — these mappers were introduced after 3.2.0b1 and
haven't been in any release, so there are no serialized DAGs with the old names
in the wild. No backward-compat aliases needed.
--
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]
