zwangsheng commented on PR #5206: URL: https://github.com/apache/kyuubi/pull/5206#issuecomment-1697001470
Thanks for your info @pan3793 IMO, pod name as a unique representation (under namespace), and deleting with pod name is less expensive than deleting with label. The api server is known to be unstable if there are too many concurrent requests. We can delete with pod name first, if pod name deletion fails, we can try to delete with label. cc @turboFei And as for your concern, appInfoStore gets the app information from the kubernetes cluster, not when kyuubi instance creates the engine, so in theory, all kyuubi instance will be consistent (with maybe a little latency, depend on informer). -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
