fpiped commented on issue #73285:
URL: https://github.com/apache/airflow/issues/73285#issuecomment-5714714805

   Thanks @AbarnaaSree, good to hear it is useful to you as well.
   
   I opened this intending to implement it myself: I have the repo set up 
locally and an open PR touching the same file (#73211), and I was waiting for a 
maintainer to weigh in on the API shape before writing any code, since this 
adds public API to two widely used operators.
   
   That said, I would rather work on it together than in parallel. My 
suggestion: wait for a maintainer to pick the shape (the `on_log_event` 
parameter, or a `KubernetesPodOperatorCallback`-style protocol), then split it, 
for example one of us takes the fetcher and operator wiring and the other the 
tests and documentation. If you prefer to drive the PR once the shape is 
settled, that is fine with me too and I will help and review.
   


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

Reply via email to