Uploaded image for project: 'Geode'
  1. Geode
  2. GEODE-7752

Update ConfigurationServiceBuilder to be more intuitive

    XMLWordPrintableJSON

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.12.0, 1.13.0
    • Component/s: configuration
    • Labels:
      None

      Description

      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.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                echobravo Ernest Burghardt
                Reporter:
                ukohlmeyer Udo Kohlmeyer
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0h
                  0h
                  Logged:
                  Time Spent - 3h
                  3h