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

(Updated May 23, 2015, 1:52 a.m.)


Review request for Ambari, Dmitro Lisnichenko, Jonathan Hurley, and Nate Cole.


Changes
-------

Correct patch this time


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


Repository: ambari


Description
-------

In HDP 2.3, Spark no longer needs Tez as a dependency.
This means that
1. Spark metainfo.xml does not need to co-host Tez-client
2. Spark History Server restart or RU restart does not need to copy tez tarball 
to HDFS. Note, this is still needed in HDP 2.2
3. Tez needs to be responsible for copying its own tarball to HDFS because its 
service check depends on it.


Diffs (updated)
-----

  
ambari-server/src/main/resources/common-services/SPARK/1.2.0.2.2/package/scripts/job_history_server.py
 106b8cc 
  
ambari-server/src/main/resources/common-services/SPARK/1.2.0.2.2/package/scripts/spark_service.py
 834e208 
  
ambari-server/src/main/resources/common-services/TEZ/0.4.0.2.1/package/scripts/service_check.py
 98973ae 
  ambari-server/src/main/resources/stacks/HDP/2.3/services/SPARK/metainfo.xml 
205ccc9 
  ambari-server/src/test/python/stacks/2.2/SPARK/test_job_history_server.py 
489037c 

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


Testing
-------

----------------------------------------------------------------------
Total run:742
Total errors:0
Total failures:0
OK

Copied changes files to a cluster with HDP 2.3.0.0 and HDFS, YARN, MR, ZK.
Then installed Spark, which did not require Tez, as expected.
Then installed Tez, whose service checked passed.
Then ran an RU and everything passed


Thanks,

Alejandro Fernandez

Reply via email to