[
https://issues.apache.org/jira/browse/DRILL-5887?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16210298#comment-16210298
]
ASF GitHub Bot commented on DRILL-5887:
---------------------------------------
Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/998#discussion_r145569034
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/DrillRoot.java
---
@@ -85,21 +86,33 @@ public ClusterInfo getClusterInfoJSON() {
// For all other cases the user info need-not or should-not be
displayed
OptionManager optionManager = work.getContext().getOptionManager();
final boolean isUserLoggedIn = AuthDynamicFeature.isUserLoggedIn(sc);
+ final String processUser = isUserLoggedIn ?
+ ImpersonationUtil.getProcessUserName() : null;
--- End diff --
Do we want to display the process user only if the user is logged in? Here,
are we assuming logged in means authenticated? Why not display the process user
otherwise?
Rather than null, should we just use a tag such as "<process user>"?
> Display process user/ groups in Drill UI
> ----------------------------------------
>
> Key: DRILL-5887
> URL: https://issues.apache.org/jira/browse/DRILL-5887
> Project: Apache Drill
> Issue Type: Bug
> Components: Client - HTTP
> Affects Versions: 1.11.0
> Reporter: Prasad Nagaraj Subramanya
> Assignee: Prasad Nagaraj Subramanya
> Priority: Minor
> Fix For: 1.12.0
>
>
> Drill UI only lists admin user/ groups specified as options
> We should display the process user/ groups who have admin privilege
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
