zwangsheng commented on PR #3852: URL: https://github.com/apache/incubator-kyuubi/pull/3852#issuecomment-1328576304
> I suppose that we could respect user defined config, whether it is `spark.master` or `spark.kubernetes.driver.master`, nonetheless, we actually do not need those configs in most cases. I understand what you said and this pr LGTM. I mean in the case we mentioned that we need to help the users set up `spark.master` (Users can't sense kubernetes master and commit using cluster deploy mode), do we need to help the user configure this `spark.kubernetes.driver.master` at the same time? -- 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]
