dstandish opened a new pull request, #45720:
URL: https://github.com/apache/airflow/pull/45720

   1. did a little work to try to make the code a little clearer in the dag 
processor where we compare versions.  Instead of current and new, which is 
confusing, I call it pre-refresh and after-refresh, essentially, which makes 
the logic a little more intelligible.  Also I provide a var `was seen` to show 
the intention of the `name in list` check.
   
   2. I propose (and do so here) renaming `latest_version` to `version` in 
bundle for reasons similar to https://github.com/apache/airflow/pull/45719.  I 
think it makes sense to think of the orm object as _itself_ representing the 
latest or current or last refreshed version. So the latest version cannot 
itself _have_ a latest version -- the latest version just has a version.  
That's sorta conceptually the issue.  In any event, by saying less, we can let 
the docs explain the nuance.
   


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

Reply via email to