----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60050/#review177866 -----------------------------------------------------------
ambari-server/src/main/java/org/apache/ambari/server/topology/PersistedStateImpl.java Lines 133-140 (patched) <https://reviews.apache.org/r/60050/#comment251576> This bit should be factored out into a separate method which than can commonly used from ClustersImpl as well. - Sebastian Toader On June 13, 2017, 6:22 p.m., Attila Doroszlai wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/60050/ > ----------------------------------------------------------- > > (Updated June 13, 2017, 6:22 p.m.) > > > Review request for Ambari, Sandor Magyari, Sumit Mohanty, and Sebastian > Toader. > > > Bugs: AMBARI-21240 > https://issues.apache.org/jira/browse/AMBARI-21240 > > > Repository: ambari > > > Description > ------- > > Remove orphaned `topology_logical_request` and `topology_request`. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClustersImpl.java > e0a496509e9256d5ca550e7dd79ae0cd0fb5882b > > ambari-server/src/main/java/org/apache/ambari/server/topology/PersistedStateImpl.java > a8b202e31b390f483d6ed1aee470179f519388a8 > > > Diff: https://reviews.apache.org/r/60050/diff/1/ > > > Testing > ------- > > Functional test: > > * up/downscale by 1 host (verify topology request is removed) > * upscale by 2 hosts, downscale by 1 host (verify topology request is > retained) > * upscale by 2 hosts with 1 pending, downscale by 1 host (verify topology > request is retained) > * upscale by 3 hosts with 1 pending, remove pending host assignments (verify > topology request is retained) > * upscale by 3 hosts with 3 pending, remove pending host assignments (verify > topology request is removed) > * restart Ambari Server (verify no database error reported) > > Unit tests: > > ``` > Tests run: 4982, Failures: 0, Errors: 0, Skipped: 39 > ``` > > > Thanks, > > Attila Doroszlai > >
