Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/16120 )
Change subject: IMPALA-9903: Reduce Kudu openTable calls per query ...................................................................... Patch Set 9: Code-Review+2 (1 comment) Thanks for the patience with iterating on this, I think this is a really good outcome. RE: Qifan's comment, I think we mostly avoid this from FS-based tables cause that state is all loaded by the catalogd and cached, but Kudu and HBase we fetch some info directly from the storage engine during planning. We did fix some things with HBase a while back like IMPALA-8912 but there could be other inefficiency. http://gerrit.cloudera.org:8080/#/c/16120/9//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/16120/9//COMMIT_MSG@27 PS9, Line 27: In manual testing this patched reduced the Kudu GetTableSchema Niiice -- To view, visit http://gerrit.cloudera.org:8080/16120 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Iec12a5be9b30e19a123142af5453a91bd4300b63 Gerrit-Change-Number: 16120 Gerrit-PatchSet: 9 Gerrit-Owner: Grant Henke <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Qifan Chen <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]> Gerrit-Reviewer: Vihang Karajgaonkar <[email protected]> Gerrit-Comment-Date: Fri, 31 Jul 2020 19:45:06 +0000 Gerrit-HasComments: Yes
