lei-xia commented on a change in pull request #1129:
URL: https://github.com/apache/helix/pull/1129#discussion_r455879634
##########
File path: helix-core/src/main/java/org/apache/helix/ConfigAccessor.java
##########
@@ -945,23 +947,23 @@ public void setInstanceConfig(String clusterName, String
instanceName,
* replaced with the value of the same field in given config if it presents.
If there is new field
* in given config but not in current config, the field will be added into
the current config..
* The list fields and map fields will be replaced as a single entry.
- * The current Cluster config will be replaced with the given clusterConfig.
WARNING: This is not
+ * The current instanceConfig will be replaced with the given
instanceConfig. WARNING: This is not
* thread-safe or concurrent updates safe.
* *
*
* @param clusterName
* @param instanceName
- * @param instanceConfig
+ * @param newInstanceConfig
*
* @return
*/
public void updateInstanceConfig(String clusterName, String instanceName,
- InstanceConfig instanceConfig) {
- updateInstanceConfig(clusterName, instanceName, instanceConfig, false);
+ InstanceConfig newInstanceConfig) {
Review comment:
It should be okey to leave it "instanceConfig" instead of
"newInstanceConfig", otherwise, you need to rename this parameter in all other
"set" or "update" related methods to keep them consistent.
----------------------------------------------------------------
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
