Nataneljpwd commented on PR #61110:
URL: https://github.com/apache/airflow/pull/61110#issuecomment-3837340971

   @SameerMesiah97 Thank you for the review!
   
   > @Nataneljpwd
   > 
   > I rest my case. I am not fully convinced that there cannot be multiple 
pods returned in your new implementation for `find_spark_job` but I will no 
longer block this PR based on the 2 concerns I communicated.
   
   It is fine to block this PR if you do not think that there cannot be 
multiple pods returned, maybe I missed something, I do not want a PR with a 
supposed bugfix merged when there is even a slight concern the bug was not 
really fixed.
   Could you explain in what case you think multiple pods can be returned? as 
maybe I just missed a detail.
   Thank you 


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