Uploaded image for project: 'Oozie'
  1. Oozie
  2. OOZIE-1890

Make oozie-site empty and reconcile defaults between oozie-default and the code

    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.2.0
    • Component/s: core
    • Labels:
      None

      Description

      As shwetha and puru suggested in OOZIE-1888, oozie-site vs oozie-default is a point of confusion for users. We've also had issues in the past where they've had different values from each other and/or from the code's default (i.e. conf.get(PROP_NAME, DEFAULT_VALUE)).

      We should make oozie-default the only source of truth by:

      1. Putting all configuration properties in oozie-default.
      2. Making oozie-site empty; if the user wants to change a property, they can copy it out of the for-reference oozie-default.
      3. Getting rid of the code defaults. It's easy for these to be out of sync with oozie-default, leading to confusion. They aren't used anyway because oozie-default should always be there (and will now have every property)

      This will require looking through all classes to make sure we're not missing anything from oozie-default and also checking that we put the proper default value (from all 3 sources) into oozie-default. It may be nice to also reorder the properties in oozie-default alphabetically (and also do this going forward with new properties).

      Also, oozie.service.WorkflowAppService.system.libpath should be set to "/user/${user.name}/share/lib" (which is what OOZIE-1888 wanted to do).

        Attachments

        1. OOZIE-1890.1.patch
          160 kB
          seoeun
        2. OOZIE-1890.2.patch
          161 kB
          seoeun
        3. OOZIE-1890.3.patch
          176 kB
          seoeun
        4. OOZIE-1890.4.patch
          185 kB
          seoeun
        5. OOZIE-1890.5.patch
          182 kB
          seoeun
        6. OOZIE-1890.6.patch
          192 kB
          seoeun
        7. OOZIE-1890.7.patch
          192 kB
          seoeun

          Issue Links

            Activity

              People

              • Assignee:
                seoeun25 seoeun
                Reporter:
                rkanter Robert Kanter
              • Votes:
                0 Vote for this issue
                Watchers:
                6 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: