Github user Leemoonsoo commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/705#issuecomment-183990823
I think it make sense to let access authentication info through
InterpreterContext.
But i think authentication info will be more than a user name in the
future. It can have some credentials/token stuff from Authentication system
(shiro), so interpreter process can reuse it without asking authentication
again, when they're required.
So my suggestion is, create such class `AuthenticationInfo`, and pass it
into InterpreterContext instead of `String userName`, for further extension in
the future.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
