----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36388/#review91272 -----------------------------------------------------------
Ship it! Ship It! - Dmitro Lisnichenko On July 10, 2015, 5:58 a.m., Sumit Mohanty wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/36388/ > ----------------------------------------------------------- > > (Updated July 10, 2015, 5:58 a.m.) > > > Review request for Ambari, Dmitro Lisnichenko, Mahadev Konar, and Sid Wagle. > > > Bugs: AMBARI-12276 > https://issues.apache.org/jira/browse/AMBARI-12276 > > > Repository: ambari > > > Description > ------- > > Ozzie running with pig fail to renew JHS delegation token when RU because JHS > recovery is not enabled. This patch adds some extra logic to the original > patch for 12276. The new logic are: > * Do not mandate the path to be the same as > yarn.timeline-service.leveldb-state-store.path > * Do not perform check if upgrading from any version that is lower than > 2.3.0.0 > * Modified unit tests to that effect > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/checks/CheckDescription.java > 0cd5352 > > ambari-server/src/main/java/org/apache/ambari/server/checks/MapReduce2JobHistoryStatePreservingCheck.java > PRE-CREATION > > ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/params_linux.py > 77092c1 > > ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/yarn.py > df8d7ce > > ambari-server/src/main/resources/stacks/HDP/2.3/services/YARN/configuration-mapred/mapred-site.xml > PRE-CREATION > > ambari-server/src/test/java/org/apache/ambari/server/checks/MapReduce2JobHistoryStatePreservingCheckTest.java > PRE-CREATION > ambari-server/src/test/python/stacks/2.0.6/YARN/test_historyserver.py > 13d9ea9 > > Diff: https://reviews.apache.org/r/36388/diff/ > > > Testing > ------- > > Manually tested, ran relevant unit tests and automated patch tests succeeded. > > > Thanks, > > Sumit Mohanty > >
