potiuk commented on PR #30255: URL: https://github.com/apache/airflow/pull/30255#issuecomment-1490753857
Hello. Coming back to that one and the whole series of them @mhenc, @vincbeck @uranusjr and others looking at it. I just created the final step of the decoupling as discussed before - so you might see where the series of changes is going. Yoy will see that while every step is essentially "smallish" and reviewable, we achieve a very nice goal where the BaseJob becomes **just** an ORM entity and the Runners require and use the BaseJob to store their state. This is **really** needed to further decouple "running" from storing "state" and I would highly appreciate some more reviews and approvals. #30255 -> #30302 -> #30308 -> https://github.com/apache/airflow/pull/30376 I could also merge that one in one huge unreviewable PR. but I hope the split and reviewabiiliyt on the "series" of changes will make it easier to get it faster through review process. Looking forward to it (AIP-44 from few other people are pretty blocked on this one). -- 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]
