Quanlong Huang has posted comments on this change. ( http://gerrit.cloudera.org:8080/13874 )
Change subject: IMPALA-8606: Don't load table meta for GET_TABLES in local catalog mode ...................................................................... Patch Set 11: LocalIncompleteTable can only be acquired by getTableIfCached which only exists on the code path of GET_TABLES HS2 operation. It won't be used at other places. As long as getTable is called, a completely loaded table will be acquired. So having LocalIncompleteTable#isLoaded return true as same as LocalFsTable does won't cause troubles in the previous patch. Anyway, to avoid confusions or introducing bugs in the future, I fix the isLoaded implementation. -- To view, visit http://gerrit.cloudera.org:8080/13874 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ia8bbab7efdf8e629abe09d89ae3bd770e3feaccb Gerrit-Change-Number: 13874 Gerrit-PatchSet: 11 Gerrit-Owner: Quanlong Huang <[email protected]> Gerrit-Reviewer: Bharath Vissapragada <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Quanlong Huang <[email protected]> Gerrit-Comment-Date: Tue, 23 Jul 2019 18:18:55 +0000 Gerrit-HasComments: No
