mrtisttt commented on issue #5369:
URL: https://github.com/apache/kyuubi/issues/5369#issuecomment-1759101291

   @pan3793 @zhouyifan279 @cfmcgrady 
   
   Hi, I have a look at the code, it can be done, but there is a point that 
needs to be aligned. Is `kyuubi.engine.user` this parameter, I understand is a 
server level parameter, rather than session level, that is, can not be changed 
in a connected session, I do not know whether my understanding is wrong? The 
main reason for this understanding is that the new `engineSpaceUser` is still 
obtained by the original `appUser`. As long as `engineSpaceUser` is unchanged 
and it is also unchanged in zookeeper namesapce, then it must get the original 
engine when it searches for the engine. Even though `kyuubi.engine.user'` is 
session level (that is, it can be set in a connection), it is useless.
   
   I don't know if I understand wrong? If I am sure that this idea is OK, I 
will proceed to modify the code to complete this feature.
   
   If there is any mistake, I hope you can guide me, thank you very much!


-- 
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]

Reply via email to