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/02/27 23:53:27 UTC

[GitHub] [airflow] potiuk commented on issue #21821: Rename `priority_weight` attribute(s)

potiuk commented on issue #21821:
URL: https://github.com/apache/airflow/issues/21821#issuecomment-1053728218


   The `priority_weight_total` includes downstream or upstream tasks prioroties) -depending on the weight rule. This is very clear to me. TaskInstance when executing is considered to have priority based on total weight (including upstram or downstram or both priorities). 
   
   This is natural consequences that Task is a definition (static) and TaskInstance is "runtime" (i.e. it contains acual status used for scheduling). I see completely no problem with it. Not sure what the proposal is here. 
   
   @michaelmicheal  - if you have a concrete proposal please open a PR and explain what you want to improve here.
   
   I am also converting this into discussion.


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