xBis7 commented on PR #43941: URL: https://github.com/apache/airflow/pull/43941#issuecomment-2773064308
Hello @ferruzzi @ashb, the PR is ready for review. My tests are all passing. I've run them at least 10 times in a row. From what I understand, the new task sdk has removed direct db access for tasks. I've gone through most of the new code but I'm not entirely familiar with the full extend of it. There might be some db related changes in this patch that aren't needed anymore. If you see something that is redundant, please point it out. BTW, I plan to create a follow up PR for cleanup but I would like to get this merged before new breaking changes end up in main. And I think that it's quite extensive as it stands right now. -- 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]
