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

    https://github.com/apache/incubator-hawq/pull/1379#discussion_r202121358
  
    --- Diff: 
pxf/pxf-service/src/main/java/org/apache/hawq/pxf/service/UGICache.java ---
    @@ -244,10 +246,10 @@ void setCleaned() {
             }
     
             /**
    -         * @return the number of active requests using the {@link 
UserGroupInformation}.
    +         * @return true if the UGI is being referenced by a session, false 
otherwise
              */
    -        int countReferences() {
    -            return referenceCount.get();
    +        private boolean isInUse() {
    --- End diff --
    
    All function modified should be of the same type as they are being used in 
a similar fashion.


---

Reply via email to