> On March 18, 2016, 1:50 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariActionExecutionHelper.java, > > line 453 > > <https://reviews.apache.org/r/45035/diff/1/?file=1305284#file1305284line453> > > > > I think adding a new column is overkill. If EU is suspended while > > stopping services, then HDFS has not yet been prepared or restarted with > > the -rollingUpgrade option. > > > > I wonder if there's a way to detect on the python side that NameNode is > > in the middle of an upgrade and hence should be restarted with that flag. > > Could even use a marker file that gets deleted after Finalize.
I don't think it is; it's useful for querying and allows us to deliver this state information via commands to any service that needs it; not just HDFS. The marker file is the hack approach - easily can cause problems. - Jonathan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45035/#review124232 ----------------------------------------------------------- On March 18, 2016, 11:20 a.m., Jonathan Hurley wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/45035/ > ----------------------------------------------------------- > > (Updated March 18, 2016, 11:20 a.m.) > > > Review request for Ambari, Alejandro Fernandez, Nate Cole, and Robert Levas. > > > Bugs: AMBARI-15482 > https://issues.apache.org/jira/browse/AMBARI-15482 > > > Repository: ambari > > > Description > ------- > > During an upgrade, the HDFS NameNode(s) are restarted with the > {{-rollingUpgrade}} flag. However, it's possible to get to the end of an > upgrade and decide to "Finalize Later". > > This allows the cluster to run in the upgraded state before committing to the > upgrade. Full cluster control is returned via the Ambari web client. > > Administrators can then decide to restart a NameNode. Upon restarting the > NameNode, it will produce an error that it was not started with the > {{rollingUpgrade}} flag. > > It seems that as long as an upgrade has not been finalized, the NameNode(s) > must be started with the {{rollingUpgrade}} to allow them to function > properly. > > STR: > - Perform a rolling upgrade from HDP 2.2 to 2.3 (or 2.3 to 2.4); as long as > there is a major version change. > - Before finalization, say "Finalize Later". All services should be up and > green. > - Now restart a NameNode > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/agent/ExecutionCommand.java > 9ea541e > > ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariActionExecutionHelper.java > 4ef215c > > ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariCustomCommandExecutionHelper.java > 303f3a4 > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UpgradeResourceProvider.java > 07061e1 > > ambari-server/src/main/java/org/apache/ambari/server/orm/entities/UpgradeEntity.java > 3f1a52b > ambari-server/src/main/java/org/apache/ambari/server/state/Cluster.java > ed3c772 > > ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClusterImpl.java > 1c7ff61 > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog222.java > 84bb9f3 > ambari-server/src/main/resources/Ambari-DDL-Derby-CREATE.sql a85202d > ambari-server/src/main/resources/Ambari-DDL-MySQL-CREATE.sql 9b4810c > ambari-server/src/main/resources/Ambari-DDL-Oracle-CREATE.sql cc3d197 > ambari-server/src/main/resources/Ambari-DDL-Postgres-CREATE.sql 07c786d > ambari-server/src/main/resources/Ambari-DDL-Postgres-EMBEDDED-CREATE.sql > ab6dc93 > ambari-server/src/main/resources/Ambari-DDL-SQLAnywhere-CREATE.sql 8e91fde > ambari-server/src/main/resources/Ambari-DDL-SQLServer-CREATE.sql 440ca44 > > ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/package/scripts/hdfs_namenode.py > e4c8c9c > > ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/package/scripts/namenode.py > 02905ec > > ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/package/scripts/params_linux.py > 905802f > ambari-web/app/controllers/main/admin/stack_and_upgrade_controller.js > 2dceccc > ambari-web/app/utils/ajax/ajax.js 29d0715 > > Diff: https://reviews.apache.org/r/45035/diff/ > > > Testing > ------- > > Pending... > > > Thanks, > > Jonathan Hurley > >