[GitHub] [iotdb] Beyyes commented on a diff in pull request #8872: [IOTDB-5287] Added status “Discouraged” to RegionGroup

[GitBox <gi...@apache.org>]

Beyyes commented on code in PR #8872:
URL: https://github.com/apache/iotdb/pull/8872#discussion_r1069118451


##########
confignode/src/main/java/org/apache/iotdb/confignode/manager/partition/RegionGroupStatus.java:
##########
@@ -29,12 +29,19 @@ public enum RegionGroupStatus {
    */
   Available("Available"),
 
+  /**
+   * All Regions in RegionGroup are in the Running, Unknown or ReadOnly status, and at least 1 node
+   * is in ReadOnly status, the number of Regions in the Unknown or ReadOnly status is less than
+   * half
+   */
+  Discouraged("Discouraged"),
+
   /**
    * The following cases will lead to Disabled RegionGroup:
    *
-   * <p>1. There is a Region in ReadOnly or Removing status
+   * <p>1. There is a Region in Removing status

Review Comment:
   Maybe not exactly, if user is migrating one region, is the RegionGroup Disabled?



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@iotdb.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org