-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/34945/
-----------------------------------------------------------
(Updated June 3, 2015, 8:20 a.m.)
Review request for Ambari, Jonathan Hurley, Sumit Mohanty, and Vitalyi
Brodetskyi.
Changes
-------
Published patch which addresses Jonathan's comments
Bugs: AMBARI-11611
https://issues.apache.org/jira/browse/AMBARI-11611
Repository: ambari
Description
-------
Now Ambari always use 1 as the value. This will render Storm-HA to not work out
of the box. We want to find the formula.
So these 2 entries should be deleted.
topology.max.replication.wait.time.sec.default : 60
topology.min.replication.count.default : 1
Fix failing unit test
Diffs (updated)
-----
ambari-server/src/main/resources/common-services/STORM/0.9.1.2.1/package/scripts/params_linux.py
4e49e2f
Diff: https://reviews.apache.org/r/34945/diff/
Testing
-------
[INFO] Rat check: Summary of files. Unapproved: 0 unknown: 0 generated: 0
approved: 136 licence.
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO]
[INFO] Ambari Views ...................................... SUCCESS [1.718s]
[INFO] Ambari Metrics Common ............................. SUCCESS [0.748s]
[INFO] Ambari Server ..................................... SUCCESS [49.324s]
[INFO] Ambari Agent ...................................... SUCCESS [18.685s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 1:11.292s
[INFO] Finished at: Tue Jun 02 17:49:39 EEST 2015
[INFO] Final Memory: 48M/482M
[INFO] ------------------------------------------------------------------------
Thanks,
Dmitro Lisnichenko
