> On Sept. 6, 2017, 8:22 p.m., Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/state/RepositoryType.java > > Lines 53-54 (patched) > > <https://reviews.apache.org/r/62128/diff/3/?file=1816613#file1816613line53> > > > > PATCH, MAINT ?
HAH! Nice catch - it was a copy / paste error that I just caught with a UT. Fixed. - Jonathan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62128/#review184761 ----------------------------------------------------------- On Sept. 6, 2017, 2:34 p.m., Jonathan Hurley wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/62128/ > ----------------------------------------------------------- > > (Updated Sept. 6, 2017, 2:34 p.m.) > > > Review request for Ambari, Dmytro Grinenko, Dmitro Lisnichenko, and Nate Cole. > > > Bugs: AMBARI-21894 > https://issues.apache.org/jira/browse/AMBARI-21894 > > > Repository: ambari > > > Description > ------- > > In the current PU UI, the revert button is shown for any PATCH type > repository. The backend should compute whether a version can be reverted. > > The backed should return a value, such as > {{ClusterStackVersions/supports_revert}} which clients, such as the web > client, can use. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClusterStackVersionResourceProvider.java > f96f257f9e > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UpgradeResourceProvider.java > 722ec0f4d2 > > ambari-server/src/main/java/org/apache/ambari/server/orm/dao/UpgradeDAO.java > 716f5b5221 > > ambari-server/src/main/java/org/apache/ambari/server/orm/entities/UpgradeEntity.java > 7f4824f6f1 > > ambari-server/src/main/java/org/apache/ambari/server/serveraction/upgrades/AbstractUpgradeServerAction.java > e012dacb17 > > ambari-server/src/main/java/org/apache/ambari/server/serveraction/upgrades/FinalizeUpgradeAction.java > 662dfef15c > > ambari-server/src/main/java/org/apache/ambari/server/state/RepositoryType.java > 2e95c59b97 > > ambari-server/src/main/java/org/apache/ambari/server/state/UpgradeContext.java > 60f44bbe85 > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog260.java > 5d50f29533 > ambari-server/src/main/resources/Ambari-DDL-Derby-CREATE.sql 2e546dcee2 > ambari-server/src/main/resources/Ambari-DDL-MySQL-CREATE.sql fefe1dc91e > ambari-server/src/main/resources/Ambari-DDL-Oracle-CREATE.sql 35866da9db > ambari-server/src/main/resources/Ambari-DDL-Postgres-CREATE.sql 1dce2ce7d4 > ambari-server/src/main/resources/Ambari-DDL-SQLAnywhere-CREATE.sql > 5627cc18f6 > ambari-server/src/main/resources/Ambari-DDL-SQLServer-CREATE.sql 07948364c5 > > ambari-server/src/test/java/org/apache/ambari/server/controller/internal/UpgradeResourceProviderTest.java > 52f41ca295 > > ambari-server/src/test/java/org/apache/ambari/server/state/UpgradeContextTest.java > 500d2297dc > > ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog260Test.java > d8e8171115 > > > Diff: https://reviews.apache.org/r/62128/diff/3/ > > > Testing > ------- > > Tests run: 4838, Failures: 0, Errors: 0, Skipped: 35 > [INFO] > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 28:54 min > [INFO] Finished at: 2017-09-06T15:37:40-04:00 > [INFO] Final Memory: 58M/1207M > [INFO] > ------------------------------------------------------------------------ > > > Thanks, > > Jonathan Hurley > >
