Uploaded image for project: 'Streams'
  1. Streams
  2. STREAMS-527

Move default values on configuration beans from json schemas into typesafe reference.conf files

VotersWatch issueWatchersLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Task
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 0.5.1
    • None
    • None

    Description

      A problem is introduced when merging configuration objects when the default values are initialized on object creation. Specifically, it becomes much harder to create partial configurations to merge, as every instance of the configuration class contains the default values across most fields.

      Typesafe config provides a recommended method for managing default values by putting a reference.conf in each module containing defaults.

      Going this route, creating a streams configuration bean with StreamsConfigurator or a component configuration bean with ComponentConfigurator will work exactly as before, and create a bean with a constructor will contain only the fields that are explicitly set by user code, ensuring that merging an ordered list of beans with juneau PojoMerge or similar methods will do exactly what is asked instead of messing up on fields with defined defaults.

      Attachments

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            steveblackmon Steve Blackmon
            steveblackmon Steve Blackmon
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Time Tracking

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

                Slack

                  Issue deployment