----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52159/#review149992 -----------------------------------------------------------
Ship it! Ship It! - Dmitro Lisnichenko On Sept. 22, 2016, 3:13 p.m., Andrew Onischuk wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/52159/ > ----------------------------------------------------------- > > (Updated Sept. 22, 2016, 3:13 p.m.) > > > Review request for Ambari and Dmitro Lisnichenko. > > > Bugs: AMBARI-18445 > https://issues.apache.org/jira/browse/AMBARI-18445 > > > Repository: ambari > > > Description > ------- > > When SmartSense is enabled, smart sense should choose default hosts for > Activity Analyzer and Activity Explorer like it does for other components. > Currently its starts by showing the error (refer SmartSenseError1.png). The > popup showing after that is also not intuitive (between 0 and 1 installed?). I > think it should choose default nodes and let user modify it if required. > > > Diffs > ----- > > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/stack_advisor.py > 7aeda19 > ambari-server/src/test/python/stacks/2.0.6/common/test_stack_advisor.py > a70922f > ambari-server/src/test/python/stacks/2.5/common/test_stack_advisor.py > 0ed1761 > > Diff: https://reviews.apache.org/r/52159/diff/ > > > Testing > ------- > > mvn clean test > > > Thanks, > > Andrew Onischuk > >