You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@helix.apache.org by "Kanak Biscuitwala (JIRA)" <ji...@apache.org> on 2013/09/18 19:36:51 UTC

[jira] [Commented] (HELIX-98) clean up setting constraint api

    [ https://issues.apache.org/jira/browse/HELIX-98?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13770994#comment-13770994 ] 

Kanak Biscuitwala commented on HELIX-98:
----------------------------------------

Resolving this by adding addStateConstraint and addTransitionConstraint to the ClusterConfig builder for the new API. Also added new builders for state and message types that wrap the existing builder.
                
> clean up setting constraint api
> -------------------------------
>
>                 Key: HELIX-98
>                 URL: https://issues.apache.org/jira/browse/HELIX-98
>             Project: Apache Helix
>          Issue Type: Sub-task
>          Components: helix-core, helix-webapp-admin
>            Reporter: Zhen Zhang
>            Assignee: Kanak Biscuitwala
>             Fix For: 0.6.2-incubating
>
>
> currently setting constraint is doing through the following api:
> 1) constraint builder
> builder.addConstraintAttribute(key1, value1).addConstraintAttribute(key2, value2)....
> 2) add constraint to cluster
> admin.addConstraint(clusterName, constraintType, constraintId, constraint)
> will be better to have:
> 1) builder(id).forMsgType(msgType).forInstance(instanceName).setConstraint(contraintValue)
> 2) add constraint to cluster
> admin.addMsgConstraint(clusterName, constraint)

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira