----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23472/#review47801 -----------------------------------------------------------
core/src/main/java/org/apache/oozie/service/ConfigurationService.java <https://reviews.apache.org/r/23472/#comment84016> Overriding configuration with system property core/src/main/java/org/apache/oozie/servlet/BaseAdminServlet.java <https://reviews.apache.org/r/23472/#comment84017> Can you fix oozie base url in SLAEmailEventListener also? core/src/main/java/org/apache/oozie/util/ZKUtils.java <https://reviews.apache.org/r/23472/#comment84018> Did you mean to get value of oozie.http.hostname as default instead of just "oozie.http.hostname" ? docs/src/site/twiki/AG_Install.twiki <https://reviews.apache.org/r/23472/#comment84020> Change the value of =OOZIE_BASE_URL= in oozie-env.sh or oozie-site.xml to point to the loadbalancer or virtual IP. The environmental variable takes precedence over the value in oozie-site.xml. For example: .... Keep both the examples (export and oozie-site.xml setting) with an (or) in between docs/src/site/twiki/AG_Install.twiki <https://reviews.apache.org/r/23472/#comment84022> can also be configured as oozie.instance.id in oozie-site.xml. The environmental variable takes precedence over oozie-site.xml. - Rohini Palaniswamy On July 15, 2014, 8:11 a.m., Purshotam Shah wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/23472/ > ----------------------------------------------------------- > > (Updated July 15, 2014, 8:11 a.m.) > > > Review request for oozie. > > > Bugs: OOZIE-1915 > https://issues.apache.org/jira/browse/OOZIE-1915 > > > Repository: oozie-git > > > Description > ------- > > Move system properties to conf properties > > > Diffs > ----- > > core/src/main/java/org/apache/oozie/service/ConfigurationService.java > 798782e > core/src/main/java/org/apache/oozie/servlet/BaseAdminServlet.java 128347e > core/src/main/java/org/apache/oozie/util/ConfigUtils.java cdf6b48 > core/src/main/java/org/apache/oozie/util/ZKUtils.java 74f6266 > core/src/main/resources/oozie-default.xml b38801f > docs/src/site/twiki/AG_Install.twiki 8b0d110 > > Diff: https://reviews.apache.org/r/23472/diff/ > > > Testing > ------- > > > Thanks, > > Purshotam Shah > >
