ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jonathan Hurley" <jhur...@hortonworks.com>
Subject Review Request 29858: Manual Upgrade Tasks Should Be More Explicit And Parameterized
Date Tue, 13 Jan 2015 18:55:28 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29858/
-----------------------------------------------------------

Review request for Ambari, Alejandro Fernandez and Nate Cole.


Bugs: AMBARI-9109
    https://issues.apache.org/jira/browse/AMBARI-9109


Repository: ambari


Description
-------

For Hive upgrade, the following manual step was shown:
{noformat}
Run the SQL file at /usr/hdp/$version/hive/scripts/metastore/upgrade to update the Hive Metastore
schema.
{noformat}

This is too vague.  I could not figure out what to do.
1. It should tell the user exactly which host this needs to be done on.
2. $version should be substituted with the actual directory name.
3. There are 40+ SQL files in the aforementioned "upgrade" folder.  What's the exact command
that I need to run?

All services that use Manual Tasks as part of the upgrade should be revisited once the parameterization
code is completed to ensure that the messages are correct. These include: Hive, Falcon, Oozie,
HDFS


Diffs
-----

  ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UpgradeGroupResourceProvider.java
4df0f9c 
  ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UpgradeItemResourceProvider.java
46d0dd4 
  ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UpgradeResourceProvider.java
b8d8502 
  ambari-server/src/main/java/org/apache/ambari/server/stack/MasterHostResolver.java b453362

  ambari-server/src/main/java/org/apache/ambari/server/state/ConfigHelper.java 5882e3e 
  ambari-server/src/main/java/org/apache/ambari/server/state/UpgradeHelper.java ac30762 
  ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/upgrade-2.2.xml c110a0b 
  ambari-server/src/test/java/org/apache/ambari/server/state/UpgradeHelperTest.java 53505f2

  ambari-server/src/test/resources/stacks/HDP/2.1.1/upgrades/upgrade_bucket_test.xml b777d94

  ambari-server/src/test/resources/stacks/HDP/2.1.1/upgrades/upgrade_test.xml aee1fcf 

Diff: https://reviews.apache.org/r/29858/diff/


Testing
-------

New tests written, full tests pending...


Thanks,

Jonathan Hurley


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message