jiajunwang commented on a change in pull request #851: Add customized view
computation and modify Helix generic controller to include new stage
URL: https://github.com/apache/helix/pull/851#discussion_r388613940
##########
File path: helix-core/src/main/java/org/apache/helix/PropertyPathBuilder.java
##########
@@ -373,6 +388,10 @@ public static String resourceConfig(String clusterName) {
return String.format("/%s/CONFIGS/RESOURCE", clusterName);
}
+ public static String customizedStateAggregationConfig(String clusterName) {
+ return String.format("/%s/CONFIGS/CUSTOMIZED_STATE_AGGREGATION",
clusterName);
Review comment:
I think it is possible that we have other configs for the customized state
except for AGGREGATION. It would be easier for us to extend if we just call it
CUSTOMIZED_STATE, which means all CUSTOMIZED_STATE configuration.
----------------------------------------------------------------
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:
us...@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@helix.apache.org
For additional commands, e-mail: reviews-h...@helix.apache.org
