Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/incubator-livy/pull/107#discussion_r219769059
--- Diff: server/src/main/scala/org/apache/livy/server/LivyServer.scala ---
@@ -115,6 +117,16 @@ class LivyServer extends Logging {
error("Failed to run kinit, stopping the server.")
sys.exit(1)
}
+ // This is and should be the only place where a login() on the UGI
is performed.
--- End diff --
Actually this wouldn't really work as - for instance -
`UserGroupInformation.loginUserFromKeytabAndReturnUGI` is ok to be called, as
it doesn't perform a `UserGroupInformation.login()`. We can list all the APIs
which we should not call, but I am not sure it is worth.
---
