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

Update ConfigurationServiceBuilder to be more intuitive

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 1.12.0, 1.13.0
    • configuration
    • 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

              echobravo Ernest Burghardt
              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