potiuk commented on PR #40568: URL: https://github.com/apache/airflow/pull/40568#issuecomment-2211661299
> Then, we can just convert to json and get the pod status with `data["status"]["phase"]` > In my testing this results in 40x speedup, so presumably, much less CPU intensive. Nice! Good one @dstandish ! > The context here is looking for pod status to change. So a cache would be unhelpful here. I am actually quite surprised to see that code. I don't think it makes a lot of sense. Pretty much whenever we call read pod we're doing it for a reason. I quite agree here. -- 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]
