Bharath Vissapragada has uploaded a new patch set (#7). Change subject: IMPALA-2347: Reuse metastore client connections in Catalog ......................................................................
IMPALA-2347: Reuse metastore client connections in Catalog Currently we create a new connection to metastore every time Catalog connects to HMS. This was intentionally done to circumvent HIVE-5181. Given it is fixed already in Hive, this patch intends to refactor the HMS client usage on the catalog to reuse the connections. Additionally this patch makes MetaStoreClient implement AutoCloseable interface and hence all the callers can use the try-with-resources to create a new metastore client and needn't explicitly call release(). Also, this patch increases the default initial metastore pool size to 10 from a previous value of 5, which is less even for a decent DDL load. Change-Id: I517c0e1efef2584cd8d34017b33574f2ad69bd52 --- M fe/src/main/java/com/cloudera/impala/analysis/CreateTableAsSelectStmt.java M fe/src/main/java/com/cloudera/impala/catalog/Catalog.java M fe/src/main/java/com/cloudera/impala/catalog/CatalogServiceCatalog.java M fe/src/main/java/com/cloudera/impala/catalog/ImpaladCatalog.java M fe/src/main/java/com/cloudera/impala/catalog/MetaStoreClientPool.java M fe/src/main/java/com/cloudera/impala/catalog/TableLoader.java M fe/src/main/java/com/cloudera/impala/service/CatalogOpExecutor.java M fe/src/test/java/com/cloudera/impala/catalog/CatalogTest.java 8 files changed, 73 insertions(+), 153 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala refs/changes/81/3381/7 -- To view, visit http://gerrit.cloudera.org:8080/3381 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I517c0e1efef2584cd8d34017b33574f2ad69bd52 Gerrit-PatchSet: 7 Gerrit-Project: Impala Gerrit-Branch: cdh5-trunk Gerrit-Owner: Bharath Vissapragada <[email protected]> Gerrit-Reviewer: Bharath Vissapragada <[email protected]> Gerrit-Reviewer: Dimitris Tsirogiannis <[email protected]> Gerrit-Reviewer: Huaisi Xu <[email protected]>
