Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-5420: Skip ACL fetch if the acl bit is not set. ......................................................................
Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/7071/1/fe/src/main/java/org/apache/impala/util/FsPermissionChecker.java File fe/src/main/java/org/apache/impala/util/FsPermissionChecker.java: Line 287: FileStatus fileStatus = fs.getFileStatus(path); > Oh nvm, I see we need to do this one any way. Yep, also another thing to note is that we want to avoid try{} catch{} since the exceptions are creating un-necessary memory churn. -- To view, visit http://gerrit.cloudera.org:8080/7071 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I96ff681fe2dddb7b777758cc49a66cc5eb741b4d Gerrit-PatchSet: 1 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Bharath Vissapragada <[email protected]> Gerrit-Reviewer: Alex Behm <[email protected]> Gerrit-Reviewer: Bharath Vissapragada <[email protected]> Gerrit-Reviewer: Dimitris Tsirogiannis <[email protected]> Gerrit-HasComments: Yes
