[hbase] branch branch-2.2 updated: HBASE-24365 MetricsTableWrapperAggregateImpl runnable fails due to exception and never runs (#1729)

[ramkrishna]

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

ramkrishna pushed a commit to branch branch-2.2
in repository https://gitbox.apache.org/repos/asf/hbase.git


The following commit(s) were added to refs/heads/branch-2.2 by this push:
     new 7cd8abf  HBASE-24365 MetricsTableWrapperAggregateImpl runnable fails 
due to exception and never runs (#1729)
7cd8abf is described below

commit 7cd8abf1a9debb7bd9a141853042fdc6f24972b3
Author: ramkrish86 <ram_krish...@hotmail.com>
AuthorDate: Mon May 18 15:11:02 2020 +0530

    HBASE-24365 MetricsTableWrapperAggregateImpl runnable fails due to 
exception and never runs (#1729)
    
    Signed-off-by: binlijin <binli...@gmail.com>
    Signed-off by: Viraj Jasani <vjas...@apache.org>
---
 .../regionserver/MetricsTableWrapperAggregateImpl.java  | 17 ++++++++++++-----
 1 file changed, 12 insertions(+), 5 deletions(-)

diff --git 
a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/MetricsTableWrapperAggregateImpl.java
 
b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/MetricsTableWrapperAggregateImpl.java
index 56b7598..781cd90 100644
--- 
a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/MetricsTableWrapperAggregateImpl.java
+++ 
b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/MetricsTableWrapperAggregateImpl.java
@@ -76,11 +76,18 @@ public class MetricsTableWrapperAggregateImpl implements 
MetricsTableWrapperAggr
               store.getMemStoreSize().getHeapSize() + 
store.getMemStoreSize().getOffHeapSize());
             mt.storeFileSize += store.getStorefilesSize();
             mt.referenceFileCount += store.getNumReferenceFiles();
-
-            mt.maxStoreFileAge = Math.max(mt.maxStoreFileAge, 
store.getMaxStoreFileAge().getAsLong());
-            mt.minStoreFileAge = Math.min(mt.minStoreFileAge, 
store.getMinStoreFileAge().getAsLong());
-            mt.totalStoreFileAge = 
(long)store.getAvgStoreFileAge().getAsDouble() *
-                store.getStorefilesCount();
+            if (store.getMaxStoreFileAge().isPresent()) {
+              mt.maxStoreFileAge =
+                  Math.max(mt.maxStoreFileAge, 
store.getMaxStoreFileAge().getAsLong());
+            }
+            if (store.getMinStoreFileAge().isPresent()) {
+              mt.minStoreFileAge =
+                  Math.min(mt.minStoreFileAge, 
store.getMinStoreFileAge().getAsLong());
+            }
+            if (store.getAvgStoreFileAge().isPresent()) {
+              mt.totalStoreFileAge =
+                  (long) store.getAvgStoreFileAge().getAsDouble() * 
store.getStorefilesCount();
+            }
             mt.storeCount += 1;
           }
           mt.regionCount += 1;