Bharath Vissapragada has posted comments on this change. ( http://gerrit.cloudera.org:8080/10792 )
Change subject: IMPALA-3040: Remove cache directives if a partition is dropped externally ...................................................................... Patch Set 5: Code-Review+2 (1 comment) Feel free to carry +2 after the fix and submit for GVO. http://gerrit.cloudera.org:8080/#/c/10792/5/fe/src/main/java/org/apache/impala/catalog/HdfsTable.java File fe/src/main/java/org/apache/impala/catalog/HdfsTable.java: http://gerrit.cloudera.org:8080/#/c/10792/5/fe/src/main/java/org/apache/impala/catalog/HdfsTable.java@1175 PS5, Line 1175: getName getName() does not print the db info. Use getFullName() instead. Also you can just use getFullName() instead of partition.getTable().getFullName(). We are already in the table class. -- To view, visit http://gerrit.cloudera.org:8080/10792 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Id7701a499405e961456adea63f3592b43bd69170 Gerrit-Change-Number: 10792 Gerrit-PatchSet: 5 Gerrit-Owner: Tianyi Wang <tw...@cloudera.com> Gerrit-Reviewer: Bharath Vissapragada <bhara...@cloudera.com> Gerrit-Reviewer: Tianyi Wang <tw...@cloudera.com> Gerrit-Comment-Date: Fri, 06 Jul 2018 20:28:14 +0000 Gerrit-HasComments: Yes