uranusjr commented on PR #74523:
URL: https://github.com/apache/airflow/pull/74523#issuecomment-6261858194

   I feel it’d make more sense to put the label before the task. Maybe 
something like
   
   ```java
   loaded.before(Label.of("when empty").on(reportEmpty), Label.of("when 
suspicious").on(reportSuspicious))
   ```
   
   I’m not sure if Python currently has both `a >> label >> [b, c]` and `a >> 
[label_for_b >> b, label_for_c >> c]` (label before or after the flow forks). 
If this is a thing, it also makes sense to also have
   
   ```java
   loaded.labeled("forks").before(reportEmpty, reportSuspicious)
   ```
   
   It’s more difficult to come up with a syntax for `after` (since that one is 
backwards by design), but the general idea is to keep the label text between 
the tasks to reflect its position in the rendered graph.


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