You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@helix.apache.org by lx...@apache.org on 2016/12/19 17:54:18 UTC
[8/9] helix git commit: [HELIX-649] Fix StateModelDef name is not
consistent
[HELIX-649] Fix StateModelDef name is not consistent
The StateModelDef is not consistent as the user provided since it use the ZNRecord id of StateModel
Project: http://git-wip-us.apache.org/repos/asf/helix/repo
Commit: http://git-wip-us.apache.org/repos/asf/helix/commit/4c3fc7f6
Tree: http://git-wip-us.apache.org/repos/asf/helix/tree/4c3fc7f6
Diff: http://git-wip-us.apache.org/repos/asf/helix/diff/4c3fc7f6
Branch: refs/heads/helix-0.6.x
Commit: 4c3fc7f6613e04ba85b88f5acbd6a16044331847
Parents: 1eb40c3
Author: Junkai Xue <jx...@linkedin.com>
Authored: Fri Dec 16 15:26:22 2016 -0800
Committer: Junkai Xue <jx...@linkedin.com>
Committed: Fri Dec 16 15:26:22 2016 -0800
----------------------------------------------------------------------
.../src/main/java/org/apache/helix/manager/zk/ZKHelixAdmin.java | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/helix/blob/4c3fc7f6/helix-core/src/main/java/org/apache/helix/manager/zk/ZKHelixAdmin.java
----------------------------------------------------------------------
diff --git a/helix-core/src/main/java/org/apache/helix/manager/zk/ZKHelixAdmin.java b/helix-core/src/main/java/org/apache/helix/manager/zk/ZKHelixAdmin.java
index 73f2cbb..c7fa2ae 100644
--- a/helix-core/src/main/java/org/apache/helix/manager/zk/ZKHelixAdmin.java
+++ b/helix-core/src/main/java/org/apache/helix/manager/zk/ZKHelixAdmin.java
@@ -763,7 +763,7 @@ public class ZKHelixAdmin implements HelixAdmin {
HelixDataAccessor accessor =
new ZKHelixDataAccessor(clusterName, new ZkBaseDataAccessor<ZNRecord>(_zkClient));
Builder keyBuilder = accessor.keyBuilder();
- accessor.setProperty(keyBuilder.stateModelDef(stateModel.getId()), stateModel);
+ accessor.setProperty(keyBuilder.stateModelDef(stateModelDef), stateModel);
}
@Override