----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59812/#review176936 -----------------------------------------------------------
Ship it! Ship It! - Nate Cole On June 5, 2017, 2:34 p.m., Jonathan Hurley wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/59812/ > ----------------------------------------------------------- > > (Updated June 5, 2017, 2:34 p.m.) > > > Review request for Ambari, Dmytro Grinenko, Dmitro Lisnichenko, and Nate Cole. > > > Bugs: AMBARI-21179 > https://issues.apache.org/jira/browse/AMBARI-21179 > > > Repository: ambari > > > Description > ------- > > With removal of {{cluster_version}}, the use of {{INIT}} as a state is no > longer valid. The initial states for hosts should be {{NOT_REQUIRED}}. > Services and Components will not need state {{INIT}}. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/orm/entities/ServiceComponentDesiredStateEntity.java > 84e464eb77 > > ambari-server/src/main/java/org/apache/ambari/server/state/RepositoryVersionState.java > a2e220ecac > ambari-server/src/main/java/org/apache/ambari/server/state/ServiceImpl.java > fe37a93fbb > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog300.java > f068798582 > ambari-server/src/main/resources/Ambari-DDL-Derby-CREATE.sql c5c83ab159 > ambari-server/src/main/resources/Ambari-DDL-MySQL-CREATE.sql 230ae10781 > ambari-server/src/main/resources/Ambari-DDL-Oracle-CREATE.sql 903fc9ee5c > ambari-server/src/main/resources/Ambari-DDL-Postgres-CREATE.sql 86fbc04ebd > ambari-server/src/main/resources/Ambari-DDL-SQLAnywhere-CREATE.sql > 47ea221d8a > ambari-server/src/main/resources/Ambari-DDL-SQLServer-CREATE.sql bbe90685b9 > > > Diff: https://reviews.apache.org/r/59812/diff/1/ > > > Testing > ------- > > Results : > > Tests run: 4942, Failures: 0, Errors: 0, Skipped: 35 > > > Thanks, > > Jonathan Hurley > >
