Details

    • Type: Sub-task
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.6.1, 1.7.0
    • Component/s: master, tserver
    • Labels:
      None

      Description

      The deprecation of ServerConfiguration causes two problems:

      1) eclipse is showing me ~300 warnings, so I'm missing other warnings in the noise

      2) ServerConfiguration is passed to some plug-ins (Balancer, at least), so any existing plug-ins are broken by the change.

      In the past, I would have been fine with just switching to the new class ServerConfigurationFactory but a growing user base and petabytes of existing systems gives me pause.

      Instead, I'm thinking we hoist the most popular methods to an abstract ServerConfiguration, and provide ServerConfiguationFactory as the implementation. This way the extensions continue to work.

      I'm still not sure what to do about static calls to get the SiteConfiguration.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                ecn Eric Newton
                Reporter:
                ecn Eric Newton
              • Votes:
                0 Vote for this issue
                Watchers:
                3 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 - 50m
                  50m