Daniel Becker has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/22863 )
Change subject: IMPALA-14040: Remove Kudu masters property from FeCatalog ...................................................................... IMPALA-14040: Remove Kudu masters property from FeCatalog This change removes getDefaultKuduMasterHosts from FeCatalog and makes the Kudu masters lookup bound directly to the backend config. Tests are adjusted for the new property lookup. Change-Id: Idcf31a724bb7bd00268accf21c0b997d92e9b23a Reviewed-on: http://gerrit.cloudera.org:8080/22863 Reviewed-by: Impala Public Jenkins <[email protected]> Reviewed-by: Daniel Becker <[email protected]> Tested-by: Daniel Becker <[email protected]> --- M fe/src/main/java/org/apache/impala/analysis/CreateTableStmt.java M fe/src/main/java/org/apache/impala/catalog/FeCatalog.java M fe/src/main/java/org/apache/impala/catalog/ImpaladCatalog.java M fe/src/main/java/org/apache/impala/catalog/local/LocalCatalog.java M fe/src/main/java/org/apache/impala/service/FeCatalogManager.java M fe/src/test/java/org/apache/impala/analysis/AnalyzeDDLTest.java M fe/src/test/java/org/apache/impala/analysis/AnalyzeKuduDDLTest.java M fe/src/test/java/org/apache/impala/analysis/StmtMetadataLoaderTest.java M fe/src/test/java/org/apache/impala/analysis/ToSqlTest.java M fe/src/test/java/org/apache/impala/authorization/AuthorizationStmtTest.java M fe/src/test/java/org/apache/impala/catalog/local/LocalCatalogTest.java M fe/src/test/java/org/apache/impala/planner/PlannerTestBase.java M fe/src/test/java/org/apache/impala/testutil/ImpaladTestCatalog.java 13 files changed, 37 insertions(+), 40 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved Daniel Becker: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/22863 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Idcf31a724bb7bd00268accf21c0b997d92e9b23a Gerrit-Change-Number: 22863 Gerrit-PatchSet: 5 Gerrit-Owner: Peter Rozsa <[email protected]> Gerrit-Reviewer: Daniel Becker <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Jason Fehr <[email protected]> Gerrit-Reviewer: Peter Rozsa <[email protected]>
