[phoenix] branch 4.x-HBase-1.4 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 4.x-HBase-1.4
in repository https://gitbox.apache.org/repos/asf/phoenix.git


The following commit(s) were added to refs/heads/4.x-HBase-1.4 by this push:
     new 91b18f4  PHOENIX-5666 IndexRegionObserver incorrectly updates PostIndexUpdateFailure metric
91b18f4 is described below

commit 91b18f4460bf98e87fb6d2dfa0c657d28a52c05c
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 6e12950..08a120e 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
@@ -763,7 +763,7 @@ public class IndexRegionObserver extends BaseRegionObserver {
       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;
       }