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

Ship it!



ambari-common/src/main/python/resource_management/libraries/functions/copy_tarball.py
<https://reviews.apache.org/r/34982/#comment138593>

    I think we need to use the temp directory defined in the agent's config for 
any temporary files.



ambari-common/src/main/python/resource_management/libraries/functions/copy_tarball.py
<https://reviews.apache.org/r/34982/#comment138594>

    Any reason you're writing this to a file instead of just reading the output?


- Jonathan Hurley


On June 3, 2015, 8:23 p.m., Alejandro Fernandez wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/34982/
> -----------------------------------------------------------
> 
> (Updated June 3, 2015, 8:23 p.m.)
> 
> 
> Review request for Ambari, Andrew Onischuk, Dmitro Lisnichenko, Jonathan 
> Hurley, Nate Cole, Srimanth Gunturi, and Sid Wagle.
> 
> 
> Bugs: AMBARI-11640
>     https://issues.apache.org/jira/browse/AMBARI-11640
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> The workaround is to start all services so that the CURRENT version exists, 
> and then rerun a HiveServer2 Start, Spark History Server2, or Tez Service 
> Check.
> The fix will be to detect if params.current_version is None, and if so, call 
> hdp-select versions, and use the value if only one version number is reported.
> Also, HiveServer2 Start needs to copy the tez tarball.
> 
> 
> Diffs
> -----
> 
>   
> ambari-common/src/main/python/resource_management/libraries/functions/copy_tarball.py
>  f8a2570 
>   
> ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/atlas_plugin_utils.py
>  d057c27 
>   
> ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/hive.py
>  32edf71 
>   ambari-server/src/test/python/stacks/2.0.6/HIVE/test_hive_server.py 051b8c3 
> 
> Diff: https://reviews.apache.org/r/34982/diff/
> 
> 
> Testing
> -------
> 
> Verified this worked on a brand new cluster install with HistoryServer Start, 
> which copies mapreduce.tar.gz to HDFS.
> 
> ----------------------------------------------------------------------
> Total run:747
> Total errors:0
> Total failures:0
> OK
> 
> 
> Thanks,
> 
> Alejandro Fernandez
> 
>

Reply via email to