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


Ship it!





ambari-server/src/main/resources/common-services/SPARK/1.2.0.2.2/package/scripts/params.py
 (lines 89 - 91)
<https://reviews.apache.org/r/42934/#comment177979>

    You can simplify this to
    
    spark_history_dir = 
default('/configurations/spark-defaults/spark.history.fs.logDirectory', 
"hdfs:///spark-history")


- Jonathan Hurley


On Jan. 28, 2016, 9:22 p.m., Jeff Zhang wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/42934/
> -----------------------------------------------------------
> 
> (Updated Jan. 28, 2016, 9:22 p.m.)
> 
> 
> Review request for Ambari, Alejandro Fernandez and Mahadev Konar.
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> This is bug introduced by Ambari-14561
> 
> 
> Diffs
> -----
> 
>   
> ambari-server/src/main/resources/common-services/SPARK/1.2.0.2.2/package/scripts/params.py
>  935c47e 
>   
> ambari-server/src/main/resources/stacks/HDP/2.4/services/SPARK/configuration/spark-defaults.xml
>  d8af790 
>   
> ambari-server/src/main/resources/stacks/HDP/2.4/services/SPARK/configuration/spark-thrift-sparkconf.xml
>  3b13496 
> 
> Diff: https://reviews.apache.org/r/42934/diff/
> 
> 
> Testing
> -------
> 
> Verified on HDP 2.4
> 
> 
> Thanks,
> 
> Jeff Zhang
> 
>

Reply via email to