> On Nov. 8, 2016, 3:24 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/state/stack/upgrade/HostOrderGrouping.java, > > line 179 > > <https://reviews.apache.org/r/53502/diff/3/?file=1556013#file1556013line179> > > > > I did mark it as Ship It, but I'm curious if we want to have this as a > > separate enum?
If we don't use HostOrderActionType.HOST_UPGRADE, I would use a constant string for now until we know better about what possible values of the enum are. - Duc ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53502/#review155291 ----------------------------------------------------------- On Nov. 8, 2016, 3:42 a.m., Duc Le wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/53502/ > ----------------------------------------------------------- > > (Updated Nov. 8, 2016, 3:42 a.m.) > > > Review request for Ambari, Alejandro Fernandez, Jonathan Hurley, and Nate > Cole. > > > Bugs: AMBARI-18681 > https://issues.apache.org/jira/browse/AMBARI-18681 > > > Repository: ambari > > > Description > ------- > > AMBARI-18681 Add host structured_out to manual task > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/state/stack/upgrade/HostOrderGrouping.java > b999f2f > > ambari-server/src/test/java/org/apache/ambari/server/state/UpgradeHelperTest.java > 78969e4 > > Diff: https://reviews.apache.org/r/53502/diff/ > > > Testing > ------- > > Unit tests passing. > > ... > Tests run: 28, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 209.464 sec > - in org.apache.ambari.server.state.UpgradeHelperTest > ... > Tests run: 4727, Failures: 0, Errors: 0, Skipped: 42 > > > Thanks, > > Duc Le > >
