leventov commented on a change in pull request #7154: rename maintenance mode
to decommission
URL: https://github.com/apache/incubator-druid/pull/7154#discussion_r263572699
##########
File path:
server/src/main/java/org/apache/druid/server/coordinator/CoordinatorDynamicConfig.java
##########
@@ -231,32 +231,35 @@ public int getMaxSegmentsInNodeLoadingQueue()
}
/**
- * Historical nodes list in maintenance mode. Coordinator doesn't assign new
segments on those nodes and moves
- * segments from those nodes according to a specified priority.
+ * List of historical servers to 'decommission'. Coordinator will not assign
new segments to 'decomissioning' servers,
+ * and segments will be moved away from them to be placed on 'active'
servers at the maximum rate specified by
+ * {@link
CoordinatorDynamicConfig#getDecommissioningMaxPercentOfMaxSegmentsToMove}.
*
* @return list of host:port entries
*/
@JsonProperty
- public Set<String> getHistoricalNodesInMaintenance()
+ public Set<String> getDecommissioningNodes()
{
- return historicalNodesInMaintenance;
+ return decommissioningNodes;
}
/**
- * Priority of segments from servers in maintenance. Coordinator takes
ceil(maxSegmentsToMove * (priority / 10))
- * from servers in maitenance during balancing phase, i.e.:
- * 0 - no segments from servers in maintenance will be processed during
balancing
- * 5 - 50% segments from servers in maintenance
- * 10 - 100% segments from servers in maintenance
- * By leveraging the priority an operator can prevent general nodes from
overload or decrease maitenance time
- * instead.
+ * The maximum number of segments that may be moved away from
'decommissioning' servers to non-decommissioning
Review comment:
Suggested "Determines the maximum number ..."
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
