Tianyi Wang has uploaded this change for review. ( http://gerrit.cloudera.org:8080/9947
Change subject: IMPALA-6820: Remove _impala_builtins from catalogd ...................................................................... IMPALA-6820: Remove _impala_builtins from catalogd _impala_builtins is created in the constructor of CatalogServiceCatalog. But it will then be deleted by CatalogServiceCatalog.reset(), leaving a delete log in catalogd and it will be sent to statestored. Though statestore doesn't send it to impalad, it's still a potential problem. This patch removes _impala_builtins from catalogd entirely, making it a impalad-only member. Change-Id: I166d8086db1d2920408f38dc56fe7c70a4c143a8 --- M fe/src/main/java/org/apache/impala/analysis/CastExpr.java M fe/src/main/java/org/apache/impala/analysis/Expr.java M fe/src/main/java/org/apache/impala/analysis/ExtractFromExpr.java M fe/src/main/java/org/apache/impala/analysis/FunctionCallExpr.java M fe/src/main/java/org/apache/impala/analysis/FunctionName.java M fe/src/main/java/org/apache/impala/catalog/Catalog.java M fe/src/main/java/org/apache/impala/catalog/ImpaladCatalog.java M fe/src/main/java/org/apache/impala/catalog/ScalarFunction.java M fe/src/main/jflex/sql-scanner.flex M fe/src/test/java/org/apache/impala/analysis/AnalyzeExprsTest.java 10 files changed, 34 insertions(+), 29 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/47/9947/1 -- 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: newchange Gerrit-Change-Id: I166d8086db1d2920408f38dc56fe7c70a4c143a8 Gerrit-Change-Number: 9947 Gerrit-PatchSet: 1 Gerrit-Owner: Tianyi Wang <tw...@cloudera.com>
