[phoenix] branch master updated: PHOENIX-5666 IndexRegionObserver incorrectly updates PostIndexUpdateFailure metric

[ka...@apache.org]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 07183d8  PHOENIX-5666 IndexRegionObserver incorrectly updates PostIndexUpdateFailure metric
07183d8 is described below

commit 07183d80e84a3ab525d62b8258adc7520d06cfd3
Author: Kadir <ko...@salesforce.com>
AuthorDate: Wed Jan 8 01:51:41 2020 -0800

    PHOENIX-5666 IndexRegionObserver incorrectly updates PostIndexUpdateFailure metric
---
 .../main/java/org/apache/phoenix/hbase/index/IndexRegionObserver.java   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/phoenix-core/src/main/java/org/apache/phoenix/hbase/index/IndexRegionObserver.java b/phoenix-core/src/main/java/org/apache/phoenix/hbase/index/IndexRegionObserver.java
index 2af8e1d..c4ae93d 100644
--- a/phoenix-core/src/main/java/org/apache/phoenix/hbase/index/IndexRegionObserver.java
+++ b/phoenix-core/src/main/java/org/apache/phoenix/hbase/index/IndexRegionObserver.java
@@ -769,7 +769,7 @@ public class IndexRegionObserver implements RegionObserver, RegionCoprocessor {
       ListMultimap<HTableInterfaceReference, Mutation> indexUpdates = post ? context.postIndexUpdates : context.preIndexUpdates;
       //short circuit, if we don't need to do any work
 
-      if (context == null || indexUpdates.isEmpty()) {
+      if (context == null || indexUpdates == null || indexUpdates.isEmpty()) {
           return;
       }