You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2022/01/06 02:39:07 UTC

[GitHub] [airflow] josh-fell commented on a change in pull request #20671: Allow depending to a @task_group as a whole

josh-fell commented on a change in pull request #20671:
URL: https://github.com/apache/airflow/pull/20671#discussion_r779263453



##########
File path: airflow/decorators/task_group.py
##########
@@ -92,9 +92,23 @@ def factory(*args, **kwargs):
                 *task_group_bound_args.args,
                 add_suffix_on_collision=True,
                 **task_group_bound_args.kwargs,
-            ):
+            ) as task_group:
                 # Invoke function to run Tasks inside the TaskGroup
-                return f(*args, **kwargs)
+                f_retval = f(*args, **kwargs)
+
+            # If 'f' returns a task, bind dependencies to it. Equilvanet to

Review comment:
       ```suggestion
               # If 'f' returns a task, bind dependencies to it. Equivalent to
   ```




-- 
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-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org