[jira] [Commented] (GEODE-7752) Update ConfigurationServiceBuilder to be more intuitive

["ASF subversion and git services (Jira)" <ji...@apache.org>]

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

ASF subversion and git services commented on GEODE-7752:
--------------------------------------------------------

Commit b3af99edd964eea8a3a02a0298db47a8ea3e9d67 in geode's branch refs/heads/develop from Jinmei Liao
[ https://gitbox.apache.org/repos/asf?p=geode.git;h=b3af99e ]

GEODE-7752: Delete the extra GeodeClusterManagementServiceBuilder (#4760)



> Update ConfigurationServiceBuilder to be more intuitive
> -------------------------------------------------------
>
>                 Key: GEODE-7752
>                 URL: https://issues.apache.org/jira/browse/GEODE-7752
>             Project: Geode
>          Issue Type: Improvement
>          Components: configuration
>            Reporter: Udo Kohlmeyer
>            Assignee: Udo Kohlmeyer
>            Priority: Major
>             Fix For: 1.12.0, 1.13.0
>
>          Time Spent: 3h
>  Remaining Estimate: 0h
>
> With the introduction of ClusterManagementServiceBuilder, an inadvertent confusing configuration manner was introduced. 
> In the Builder pattern, the setter methods are optional and required fields are either added on the constructor or build method.
> The current ClusterManangementServiceBuilder introduced the notion of required optionality. Where you had to pick at least one of the setters.
> To fix this, I removed `setConnectionConfig` which is really only required for the Transport and removed that option.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)