Uploaded image for project: 'Tamaya'
  1. Tamaya
  2. TAMAYA-321

Replace/Deprecate ConfigurationContextBuilder in favour of ConfigurationBuilder

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 0.3-incubating
    • 0.4-incubating
    • API
    • None

    Description

      Building a configuration is valid use case. Unfortunately as of now, a user must built a ConfigurationContext to create a Configuration:

      ConfigurationContextBuilder b = ConfigurationProvider.getConfigurationContextBuilder();
      // build up context with b
      ConfigurationContext ctx = b.build();
      Configuration cfg = ConfigurationProvider.createConfiguration(ctx);

      This is cumbersome. Whereas ConfigurationContext as a container for all the small parts, that build up a Configuration makes sense, the user basically should be able to build a configuration directly:

      ConfigurationBuilder b = ConfigurationProvider.getConfigurationBuilder();
      // build up config with b
      Configuration cfg = b.build();

      Proposal

      Add:

      • ConfigurationBuilder (same functionaly as former ConfigurationContextBuilder, but for building a Configuration).
      • ConfigurationBuilder ConfigurationProvider.getConfigurationBuilder()
      • ConfigurationBuilder ConfigurationProviderSpi#getConfigurationBuilder()
      • ConfigurationBuilder Configuration#toBuilder()

      Deprecate:

      • ConfigurationContextBuilder
      • ConfigurationContextBuilder ConfigurationProvider.getConfigurationContextBuilder()
      • ConfigurationContextBuilder ConfigurationProviderSpi#getConfigurationContextBuilder()
      • ConfigurationContextBuilder ConfigurationContext#toBuilder()

      Attachments

        Activity

          People

            anatole Anatole Tresch
            anatole Anatole Tresch
            Votes:
            1 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Time Tracking

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