dstandish commented on PR #40568: URL: https://github.com/apache/airflow/pull/40568#issuecomment-2209525174
> I don't understand the goal of this PR in its current state (replacing `self.read_pod` that supports caching by a simple K8S client call). > I don't understand the goal of this PR in its current state (replacing `self.read_pod` that supports caching by a simple K8S client call). > > I request a change to avoid merging it accidentally. @hussein-awala look here https://github.com/apache/airflow/pull/40360#issuecomment-2182270584 that is the impetus for this PR "why load the whole pod when you can just get the status" is, i think, the idea. -- 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]
