You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by da...@apache.org on 2019/03/26 21:32:05 UTC

[hive] branch master updated: HIVE-21478: Metastore cache update shall capture exception (Daniel Dai, reviewed by Zoltan Haindrich)

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 12f8371  HIVE-21478: Metastore cache update shall capture exception (Daniel Dai, reviewed by Zoltan Haindrich)
12f8371 is described below

commit 12f83719d940034dc8c6273e2772f6b30d07108e
Author: Daniel Dai <da...@gmail.com>
AuthorDate: Tue Mar 26 14:31:13 2019 -0700

    HIVE-21478: Metastore cache update shall capture exception (Daniel Dai, reviewed by Zoltan Haindrich)
    
    Signed-off-by: Zoltan Haindrich <ki...@rxd.hu>
---
 .../java/org/apache/hadoop/hive/metastore/cache/CachedStore.java    | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/cache/CachedStore.java b/standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/cache/CachedStore.java
index bded743..41b72d1 100644
--- a/standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/cache/CachedStore.java
+++ b/standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/cache/CachedStore.java
@@ -715,7 +715,11 @@ public class CachedStore implements RawStore, Configurable {
           }
         } else {
           // TODO: prewarm and update can probably be merged.
-          update();
+          try {
+            update();
+          } catch (Exception e) {
+            LOG.error("periodical refresh fail ", e);
+          }
         }
       } else {
         try {