This is an automated email from the ASF dual-hosted git repository.
apurtell pushed a commit to branch branch-1
in repository https://gitbox.apache.org/repos/asf/hbase.git
The following commit(s) were added to refs/heads/branch-1 by this push:
new 084745d HBASE-22235 OperationStatus.{SUCCESS|FAILURE|NOT_RUN} are not
visible to 3rd party coprocessors
084745d is described below
commit 084745de186402ffa6df14f70885ea68dd8cec4a
Author: Andrew Purtell <[email protected]>
AuthorDate: Mon Apr 15 10:48:08 2019 -0700
HBASE-22235 OperationStatus.{SUCCESS|FAILURE|NOT_RUN} are not visible to
3rd party coprocessors
---
.../src/main/java/org/apache/hadoop/hbase/HConstants.java | 2 +-
.../apache/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 594bb0e..99c2cc0 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 b8473d7..23e1144 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.
*
*/
[email protected]
[email protected](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;