----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32721/#review78520 -----------------------------------------------------------
Ship it! Ship It! - Vitalyi Brodetskyi On Квітень 1, 2015, 2:23 після полудня, Andrew Onischuk wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/32721/ > ----------------------------------------------------------- > > (Updated Квітень 1, 2015, 2:23 після полудня) > > > Review request for Ambari and Vitalyi Brodetskyi. > > > Bugs: AMBARI-10310 > https://issues.apache.org/jira/browse/AMBARI-10310 > > > Repository: ambari > > > Description > ------- > > This requires adding a new property with this port to storm-env. And > substiting it into supervisor.childopts. > And using this property in alerts.json, so this port is not hardcoded there. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog210.java > ba4dd2d > > ambari-server/src/main/resources/common-services/STORM/0.9.1.2.1/alerts.json > 6557df9 > > ambari-server/src/main/resources/common-services/STORM/0.9.1.2.1/configuration/storm-env.xml > c663116 > > ambari-server/src/main/resources/common-services/STORM/0.9.1.2.1/configuration/storm-site.xml > 5e96fb1 > > ambari-server/src/main/resources/common-services/STORM/0.9.1.2.1/package/scripts/params.py > 2d575b0 > > ambari-server/src/main/resources/stacks/HDP/2.2/services/STORM/configuration/storm-site.xml > c933099 > > Diff: https://reviews.apache.org/r/32721/diff/ > > > Testing > ------- > > mvn clean test > > > Thanks, > > Andrew Onischuk > >
