----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61273/#review182092 -----------------------------------------------------------
Ship it! Ship It! - Andrew Onischuk On Aug. 2, 2017, 3:08 p.m., Dmytro Sen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/61273/ > ----------------------------------------------------------- > > (Updated Aug. 2, 2017, 3:08 p.m.) > > > Review request for Ambari, Sumit Mohanty, Sid Wagle, and Vitalyi Brodetskyi. > > > Bugs: AMBARI-21621 > https://issues.apache.org/jira/browse/AMBARI-21621 > > > Repository: ambari > > > Description > ------- > > If DN is deleted the hostname is removed from exclude file and refreshnodes > command is called. If those actions are not followed by NN restart, the > deleted DN will be shown as DEAD on NN UI. > > The slave should be removed by NN from the UI, but Ambari should not call > refreshnodes command on deletion to ensure slave is in "decommissioned and > dead" state, not "dead" > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java > 3016027 > > ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/package/scripts/params_linux.py > c9df2b9 > > ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/package/scripts/params_windows.py > b3ac578 > > ambari-server/src/main/resources/common-services/HDFS/3.0.0.3.0/package/scripts/params_linux.py > 1581c2a > > ambari-server/src/main/resources/common-services/HDFS/3.0.0.3.0/package/scripts/params_windows.py > 6c492d8 > > ambari-server/src/main/resources/stacks/BIGTOP/0.8/services/HDFS/package/scripts/params.py > 9cf163a > > ambari-server/src/test/java/org/apache/ambari/server/controller/AmbariManagementControllerTest.java > 5fb236b > > > Diff: https://reviews.apache.org/r/61273/diff/3/ > > > Testing > ------- > > manual + unit tests > > > Thanks, > > Dmytro Sen > >
