turboFei commented on PR #3213:
URL: 
https://github.com/apache/incubator-kyuubi/pull/3213#issuecomment-1218873219

   Now the only remaining comments is 
https://github.com/apache/incubator-kyuubi/pull/3213#discussion_r947509152
   
   I think we can fallback to get binddn according to guidKey & baseDn if 
`AUTHENTICATION_LDAP_BINDDN` is not specified.
   
   Otherwise, the customers that use the old kyuubi version will meet failure 
when upgrading kyuubi.
   
   @dev-lpq 


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