bolkedebruin commented on code in PR #34683:
URL: https://github.com/apache/airflow/pull/34683#discussion_r1341282384
##########
airflow/serialization/serializers/timezone.py:
##########
@@ -69,4 +82,28 @@ def deserialize(classname: str, version: int, data: object)
-> Timezone:
if isinstance(data, int):
return fixed_timezone(data)
+ if "zoneinfo.ZoneInfo" in classname: # capturing backports and stdlib
+ if PY39:
+ from zoneinfo import ZoneInfo
+ else:
+ from backports.zoneinfo import ZoneInfo
Review Comment:
In serialization we want to keep imports to a minimum, so not in the root
indeed. I could check for the version explicitly like `sys.version_info >= (3,
9)`. The old fashioned stuff is slower
(https://stackoverflow.com/questions/2522005/cost-of-exception-handlers-in-python)
and in serialization the focus is on speed, particularly with datetimes.
--
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: commits-unsubscr...@airflow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
