[geode] branch develop updated: GEODE-7804: move methods up from RebalanceResult to OperationResult (#4801)

[dschneider]

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

dschneider pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/geode.git


The following commit(s) were added to refs/heads/develop by this push:
     new 9a3c62d  GEODE-7804: move methods up from RebalanceResult to 
OperationResult (#4801)
9a3c62d is described below

commit 9a3c62d51f580597c8ed26c3ee78cab26a187b38
Author: Darrel Schneider <dschnei...@pivotal.io>
AuthorDate: Thu Mar 12 16:54:47 2020 -0700

    GEODE-7804: move methods up from RebalanceResult to OperationResult (#4801)
    
    moved two methods up from RebalanceResult to OperationResult
---
 .../apache/geode/management/runtime/OperationResult.java  | 15 ++++++++++++++-
 .../apache/geode/management/runtime/RebalanceResult.java  | 11 -----------
 2 files changed, 14 insertions(+), 12 deletions(-)

diff --git 
a/geode-management/src/main/java/org/apache/geode/management/runtime/OperationResult.java
 
b/geode-management/src/main/java/org/apache/geode/management/runtime/OperationResult.java
index 59e80fc..e4b7b76 100644
--- 
a/geode-management/src/main/java/org/apache/geode/management/runtime/OperationResult.java
+++ 
b/geode-management/src/main/java/org/apache/geode/management/runtime/OperationResult.java
@@ -22,8 +22,21 @@ import 
org.apache.geode.management.api.ClusterManagementOperation;
 import org.apache.geode.management.api.JsonSerializable;
 
 /**
- * marker interface for the final result of a {@link 
ClusterManagementOperation}
+ * Describes the result of a starting or checking the status of a
+ * {@link ClusterManagementOperation}.
  */
 @Experimental
 public interface OperationResult extends JsonSerializable, Serializable {
+  /**
+   * Returns true if the operation was successful; false if it failed.
+   */
+  boolean getSuccess();
+
+  /**
+   * Returns details about what caused the operation to fail or succeed.
+   * If the operation failed then a description of what was wrong will
+   * be in this status message. If the operation was successful then
+   * the status message may contain additional information.
+   */
+  String getStatusMessage();
 }
diff --git 
a/geode-management/src/main/java/org/apache/geode/management/runtime/RebalanceResult.java
 
b/geode-management/src/main/java/org/apache/geode/management/runtime/RebalanceResult.java
index 6938d86..636cdd0 100644
--- 
a/geode-management/src/main/java/org/apache/geode/management/runtime/RebalanceResult.java
+++ 
b/geode-management/src/main/java/org/apache/geode/management/runtime/RebalanceResult.java
@@ -29,15 +29,4 @@ public interface RebalanceResult extends OperationResult {
    * Return summary of work performed for each region
    */
   List<RebalanceRegionResult> getRebalanceRegionResults();
-
-  /**
-   * May contain additional information special cases, for example when the 
rebalance operation
-   * "succeeds" only because there are no partitioned regions or only one 
server.
-   */
-  String getStatusMessage();
-
-  /**
-   * Whether the rebalance operation was overall successful or not.
-   */
-  boolean getSuccess();
 }