[iceberg] branch master updated: Hive: Use default values of client pool configuration (Laszlo Pinter reviewed by Marton Bod and Peter Vary)

[pvary]

This is an automated email from the ASF dual-hosted git repository.

pvary pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/iceberg.git


The following commit(s) were added to refs/heads/master by this push:
     new 7779953  Hive: Use default values of client pool configuration (Laszlo 
Pinter reviewed by Marton Bod and Peter Vary)
7779953 is described below

commit 7779953bd2dad7990725391f9d5d8be077a7048d
Author: László Pintér <47777102+lcspin...@users.noreply.github.com>
AuthorDate: Mon May 10 13:29:41 2021 +0200

    Hive: Use default values of client pool configuration (Laszlo Pinter 
reviewed by Marton Bod and Peter Vary)
    
    Closes #2550
---
 .../src/main/java/org/apache/iceberg/hive/HiveCatalog.java          | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git 
a/hive-metastore/src/main/java/org/apache/iceberg/hive/HiveCatalog.java 
b/hive-metastore/src/main/java/org/apache/iceberg/hive/HiveCatalog.java
index 9b8e0b4..68ff625 100644
--- a/hive-metastore/src/main/java/org/apache/iceberg/hive/HiveCatalog.java
+++ b/hive-metastore/src/main/java/org/apache/iceberg/hive/HiveCatalog.java
@@ -83,9 +83,11 @@ public class HiveCatalog extends BaseMetastoreCatalog 
implements SupportsNamespa
     this.fileIO = new HadoopFileIO(conf);
     Map<String, String> properties = ImmutableMap.of(
             CatalogProperties.CLIENT_POOL_CACHE_EVICTION_INTERVAL_MS,
-            conf.get(CatalogProperties.CLIENT_POOL_CACHE_EVICTION_INTERVAL_MS),
+            conf.get(CatalogProperties.CLIENT_POOL_CACHE_EVICTION_INTERVAL_MS,
+                    
String.valueOf(CatalogProperties.CLIENT_POOL_CACHE_EVICTION_INTERVAL_MS_DEFAULT)),
             CatalogProperties.CLIENT_POOL_SIZE,
-            conf.get(CatalogProperties.CLIENT_POOL_SIZE)
+            conf.get(CatalogProperties.CLIENT_POOL_SIZE,
+                    String.valueOf(CatalogProperties.CLIENT_POOL_SIZE_DEFAULT))
     );
     this.clients = new CachedClientPool(conf, properties);
   }