stoty edited a comment on pull request #86: URL: https://github.com/apache/phoenix-omid/pull/86#issuecomment-775750181
Let me recap what I see: The original problem was that the old code didnt't check for a pre-existing kerberos ugi, and forced logging in with the provided credentials, even if they weren't provided. This made Omid almost unusable in a secure environment, as every principal would have needed their own specialized client config, with principals and keytabs. The new code fixes it by reusing existing credentials from the cache / UGI, and for good measure replaces most of the kerberos login code which the Phoenix version. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
