You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@oozie.apache.org by "Andras Salamon (JIRA)" <ji...@apache.org> on 2019/04/03 11:37:00 UTC

[jira] [Created] (OOZIE-3462) Reconcile defaults between oozie-default.xml and the code

Andras Salamon created OOZIE-3462:
-------------------------------------

             Summary: Reconcile defaults between oozie-default.xml and the code
                 Key: OOZIE-3462
                 URL: https://issues.apache.org/jira/browse/OOZIE-3462
             Project: Oozie
          Issue Type: Improvement
            Reporter: Andras Salamon


OOZIE-1890 addressed the issues of config default values with the following steps:
 * make {{oozie-site.xml}} empty
 * put (almost) all configuration from {{oozie-default.xml}}
 * remove default values from the code

It left the following confusing text at the [beginning|https://github.com/apache/oozie/blob/master/core/src/main/resources/oozie-default.xml#L22-L25] of the oozie-default.xml site:
{noformat}
This file is in the Oozie configuration directory only for reference.
It is not loaded by Oozie, Oozie uses its own privatecopy.{noformat}
Maybe it's referring to the {{oozie-site.xml}} or assumes that all the default values are specified in the code. I think this text is misleading, we should delete it.

Although OOZIE-1890 removed all the defaults from the source code (replacing code like {{getInt(name, defaultValue)}} with {{getInt(name))}} later defaults were introduced again into the code.

I think we need to revisit the issue again:
 # remove default values from the code
 # modify {{ConfigurationService}} to avoid using default values in the code (at least make those methods deprecated, maybe throw {{IllegalArgumentException}} but I'm afraid that would be too strict.
  

 



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)