----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29237/#review65626 -----------------------------------------------------------
Ship it! Ship It! - Nate Cole On Dec. 18, 2014, 11:19 p.m., Tom Beerbower wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/29237/ > ----------------------------------------------------------- > > (Updated Dec. 18, 2014, 11:19 p.m.) > > > Review request for Ambari, Jonathan Hurley and Nate Cole. > > > Bugs: AMBARI-8811 > https://issues.apache.org/jira/browse/AMBARI-8811 > > > Repository: ambari > > > Description > ------- > > 1. The currently executing upgrade group goes back and forth from IN_PROGRESS > to PENDING state. > 2. Completed upgrade groups should have COMPLETED state instead of PENDING. > 3. When the upgrade is paused, overall progress shouldn't be at 100%. The > percentage completion should remain as is. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/RequestResourceProvider.java > 549a20d > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/StageResourceProvider.java > f2ea76d > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UpgradeGroupResourceProvider.java > 46dcd1e > > ambari-server/src/test/java/org/apache/ambari/server/controller/internal/StageResourceProviderTest.java > c97de8a > > Diff: https://reviews.apache.org/r/29237/diff/ > > > Testing > ------- > > Manual tests. > > New unit tests added. > > All pass. > > Results : > > Tests run: 2440, Failures: 0, Errors: 0, Skipped: 13 > > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 27:02 min > [INFO] Finished at: 2014-12-18T22:59:53-05:00 > [INFO] Final Memory: 34M/449M > [INFO] > ------------------------------------------------------------------------ > > > Thanks, > > Tom Beerbower > >
