Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16898 )
Change subject: IMPALA-10117: Skip calls to FsPermissionCache for blob stores ...................................................................... IMPALA-10117: Skip calls to FsPermissionCache for blob stores This avoids calling precacheChildrenOf() in cases when the cached values will never be used. This change simply skips calling precacheChildrenOf() in the cases when getPermissions() is never called. There is some opportunity to clean up this permissions checking further, but I decided to keep this fix limited in scope. Change-Id: I2034695a956307309f656d56aa57aa07ae5163d8 Reviewed-on: http://gerrit.cloudera.org:8080/16898 Reviewed-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Tested-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> --- M fe/src/main/java/org/apache/impala/catalog/HdfsTable.java 1 file changed, 27 insertions(+), 14 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/16898 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I2034695a956307309f656d56aa57aa07ae5163d8 Gerrit-Change-Number: 16898 Gerrit-PatchSet: 3 Gerrit-Owner: Tim Armstrong <tarmstr...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Quanlong Huang <huangquanl...@gmail.com> Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com> Gerrit-Reviewer: Zoltan Borok-Nagy <borokna...@cloudera.com>