Dimitris Tsirogiannis has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/9947 )

Change subject: IMPALA-6820: Remove _impala_builtins from catalogd
......................................................................


Patch Set 1: Code-Review+2

(1 comment)

http://gerrit.cloudera.org:8080/#/c/9947/1//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/9947/1//COMMIT_MSG@9
PS1, Line 9: _impala_builtins is created in the constructor of
           : CatalogServiceCatalog.
The _impala_builtins database is initialized in the constructor of Catalog and 
hence is inherited by both the CatalogServiceCatalog and ImpaladCatalog. Since  
_impala_builtins is not used by the catalog server and to avoid the overhead of 
managing this database during normal metadata operations (e.g. invalidate), it 
is moved to the ImpaladCatalog class.



--
To view, visit http://gerrit.cloudera.org:8080/9947
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I166d8086db1d2920408f38dc56fe7c70a4c143a8
Gerrit-Change-Number: 9947
Gerrit-PatchSet: 1
Gerrit-Owner: Tianyi Wang <tw...@cloudera.com>
Gerrit-Reviewer: Dimitris Tsirogiannis <dtsirogian...@cloudera.com>
Gerrit-Comment-Date: Mon, 09 Apr 2018 23:35:07 +0000
Gerrit-HasComments: Yes

Reply via email to