From c5a42dbdf335a386ee3b212da7074bc8c98f81da Mon Sep 17 00:00:00 2001 From: Andrew Purtell Date: Mon, 15 Apr 2019 10:48:08 -0700 Subject: [PATCH] HBASE-22235 OperationStatus.{SUCCESS|FAILURE|NOT_RUN} are not visible to 3rd party coprocessors --- .../java/org/apache/hadoop/hbase/HConstants.java | 2 +- .../hadoop/hbase/regionserver/OperationStatus.java | 13 ++++++------- 2 files changed, 7 insertions(+), 8 deletions(-) diff --git a/hbase-common/src/main/java/org/apache/hadoop/hbase/HConstants.java b/hbase-common/src/main/java/org/apache/hadoop/hbase/HConstants.java index 594bb0e548..99c2cc0dcf 100644 --- a/hbase-common/src/main/java/org/apache/hadoop/hbase/HConstants.java +++ b/hbase-common/src/main/java/org/apache/hadoop/hbase/HConstants.java @@ -85,7 +85,7 @@ public final class HConstants { /** * Status codes used for return values of bulk operations. */ - @InterfaceAudience.Private + @InterfaceAudience.LimitedPrivate(HBaseInterfaceAudience.COPROC) public enum OperationStatusCode { NOT_RUN, SUCCESS, diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/OperationStatus.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/OperationStatus.java index b8473d7d96..23e1144952 100644 --- a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/OperationStatus.java +++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/OperationStatus.java @@ -19,7 +19,9 @@ package org.apache.hadoop.hbase.regionserver; import org.apache.hadoop.hbase.classification.InterfaceAudience; +import org.apache.hadoop.hbase.HBaseInterfaceAudience; import org.apache.hadoop.hbase.HConstants.OperationStatusCode; + /** * * This class stores the Operation status code and the exception message @@ -28,20 +30,17 @@ import org.apache.hadoop.hbase.HConstants.OperationStatusCode; * the operation status in future. * */ -@InterfaceAudience.Private +@InterfaceAudience.LimitedPrivate(HBaseInterfaceAudience.COPROC) public class OperationStatus { /** Singleton for successful operations. */ - static final OperationStatus SUCCESS = - new OperationStatus(OperationStatusCode.SUCCESS); + public static final OperationStatus SUCCESS = new OperationStatus(OperationStatusCode.SUCCESS); /** Singleton for failed operations. */ - static final OperationStatus FAILURE = - new OperationStatus(OperationStatusCode.FAILURE); + public static final OperationStatus FAILURE = new OperationStatus(OperationStatusCode.FAILURE); /** Singleton for operations not yet run. */ - static final OperationStatus NOT_RUN = - new OperationStatus(OperationStatusCode.NOT_RUN); + public static final OperationStatus NOT_RUN = new OperationStatus(OperationStatusCode.NOT_RUN); private final OperationStatusCode code; -- 2.21.0