mrtisttt commented on PR #5653: URL: https://github.com/apache/kyuubi/pull/5653#issuecomment-1805362438
> > When implementing it in beeline in the future, at the API level, does it also need to simplify the name to `proxyUser` instead of `kyuubi.session.proxy.user`? > > I think we actually should change in JDBC driver module actually. When in Rome do as the Romans do. Just follow the existing parameter style. > > For RESTful API, `proxyUser` should be a good parameter name, even the current API does not follow that style. (we may want to design RESTful API v2 some day, after we learn something from the v1 practice) Well, I get it. For RESTful API, I have changed it to `proxyUser`. As for the JDBC level, I will look at how to implement it in the future. -- 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]
