tvalentyn commented on code in PR #35725: URL: https://github.com/apache/beam/pull/35725#discussion_r2311270981
########## sdks/python/apache_beam/coders/coders.py: ########## @@ -940,6 +979,31 @@ def to_type_hint(self): return Any +def _should_force_use_dill(): + from apache_beam.coders import typecoders + from apache_beam.transforms.util import is_v1_prior_to_v2 + + if not is_v1_prior_to_v2(v1=typecoders.registry.update_compatibility_version, Review Comment: i would probably do the None check here rather than rely on the helper, since semantics of comparing a version with None is a bit debatable. feel free to keep as is though -- 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: github-unsubscr...@beam.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org