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.
---