----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37437/#review95291 -----------------------------------------------------------
ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/upgrade-2.3.xml (line 580) <https://reviews.apache.org/r/37437/#comment150154> Di, can you explain this some more? This is performing an RU from HDP 2.2 -> 2.3, and that property exists in HDP 2.3, so due to config merging, it means that if the property does not exist or the user has not modified it, it will be set to *...HistoryServerLeveldbStateStoreService. Note: if the user does modify it, their value will be preserved. From your description, it sounds like the property does not work in HDP 2.2, so why should there be a PreCheck for it at all? - Alejandro Fernandez On Aug. 13, 2015, 1:13 p.m., Di Li wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/37437/ > ----------------------------------------------------------- > > (Updated Aug. 13, 2015, 1:13 p.m.) > > > Review request for Ambari and Alejandro Fernandez. > > > Bugs: AMBARI-12751 > https://issues.apache.org/jira/browse/AMBARI-12751 > > > Repository: ambari > > > Description > ------- > > RU precheck requires "mapreduce.jobhistory.recovery.store.class" property to > be set in the mapred-site.xml with value > "org.apache.hadoop.mapreduce.v2.hs.HistoryServerLeveldbStateStoreService" > > With this value set, the HDP 2.2 level MapRed history server can not start. > It shows an err about class > org.apache.hadoop.mapreduce.v2.hs.HistoryServerLeveldbStateStoreService does > not exist. > > When downgrading, the mapred properties added to pass the RU precheck needs > to be removed. > > > Diffs > ----- > > ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/upgrade-2.3.xml > f6823c8 > > Diff: https://reviews.apache.org/r/37437/diff/ > > > Testing > ------- > > manually patch cluster to run rolling upgrade, after it passes the Core > Master step (which includes upgrading the MapRed2 History server), run > downgrade, verify the MapRed2 history server is still running once the > downgrade is done, the three mapred properties added to satisfy the ru > precheck are removed. > > > Thanks, > > Di Li > >
