Ramit Kataria

Results 8 comments of Ramit Kataria

Good idea, I'll run this soon

@Bowrna I ran this through the system tests infra on local executor and most of the tests are passing but there are a few failures, probably not related to this...

Also, could you please update the branch with main? There have been changes recently that fixed most of the tests

> > Also, could you please update the branch with main? There have been changes recently that fixed most of the tests > > @ramitkataria This is done. Thanks, I'll...

@Bowrna Tests are complete and there were no additional failures so we should be good to merge!

@ashb @uranusjr @pierrejeambrun I've addressed all the unresolved comments so I'm looking for a re-review

I just briefly looked at callback related changes but will try to take a more thorough look later if I get a chance

I believe option 1 should be relatively straightforward to implement once support for `ExecuteCallback` is added to workers, which is in progress https://github.com/apache/airflow/blob/25202a8a703e149ffaec9790112a5b11f605c732/airflow-core/src/airflow/executors/workloads.py#L146