[GitHub] incubator-hawq pull request #1379: HAWQ-1622. Cache PXF proxy UGI so that cl...

[denalex]

Github user denalex commented on a diff in the pull request:

    https://github.com/apache/incubator-hawq/pull/1379#discussion_r201519410
  
    --- Diff: 
pxf/pxf-service/src/main/java/org/apache/hawq/pxf/service/servlet/SecurityServletFilter.java
 ---
    @@ -89,32 +106,49 @@ public Boolean run() throws IOException, 
ServletException {
                 };
     
                 // create proxy user UGI from the UGI of the logged in user 
and execute the servlet chain as that user
    -            UserGroupInformation proxyUGI = null;
    +            UGICacheEntry timedProxyUGI = cache.getTimedProxyUGI(session);
    --- End diff --
    
    name it proxyUGIEntry ? since actual UGI is a member of that object


---