potiuk commented on code in PR #53450:
URL: https://github.com/apache/airflow/pull/53450#discussion_r2217304558


##########
task-sdk/src/airflow/sdk/definitions/taskgroup.py:
##########
@@ -669,36 +672,41 @@ def iter_mapped_dependencies(self) -> Iterator[Operator]:
             yield op
 
 
-def task_group_to_dict(task_item_or_group):
+@cache

Review Comment:
   This cache does not cache the "result" of the callable - it caches callable 
itself. So what you get as a result is th the right callable depending on the 
configuration. And you need ot restart airflow to reload configuration. So it's 
ok.



-- 
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]

Reply via email to