potiuk commented on code in PR #36147:
URL: https://github.com/apache/airflow/pull/36147#discussion_r1426814072
##########
airflow/serialization/serde.py:
##########
@@ -360,7 +360,7 @@ def _register():
@functools.lru_cache(maxsize=None)
def _get_patterns() -> list[Pattern]:
patterns = conf.get("core", "allowed_deserialization_classes").split()
- return [re2.compile(re2.sub(r"(\w)\.", r"\1\..", p)) for p in patterns]
+ return [re2.compile(re2.sub(r"(\w)\.?\*", r"\1\.", p)) for p in patterns]
Review Comment:
Just one think - make sure to add a `significant` newstfragment for it (see
`newsfragments` folder) - this change will likely break someone's workflows, so
we need to give the users some hint on how they shoudl fix the problems they
might have (if they do). I think it's rather unlikely it will affect someone's
deployment, so that's why bugfix, but we should have answer ready for users if
it does @tobiaszorzetto
--
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
