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


Ship it!




Ship It!

- Di Li


On Dec. 19, 2016, 9:24 p.m., Tim Thorpe wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/54874/
> -----------------------------------------------------------
> 
> (Updated Dec. 19, 2016, 9:24 p.m.)
> 
> 
> Review request for Ambari, Alejandro Fernandez, Andrew Onischuk, Di Li, 
> Jonathan Hurley, Jayush Luniya, and Sumit Mohanty.
> 
> 
> Bugs: AMBARI-19241
>     https://issues.apache.org/jira/browse/AMBARI-19241
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Currently the hdfs_resources.py supports
> 1) create files/directories in HDFS
> 2) delete files/directories in HDFS
> 3) upload files/directories to HDFS
> 
> We should also support download of files/directories from HDFS. This will 
> help particularly in cloud environments where the users of the cluster don't 
> necessarily have write access to the local file system.
> 
> 
> Diffs
> -----
> 
>   
> ambari-common/src/main/python/resource_management/libraries/functions/download_from_hdfs.py
>  PRE-CREATION 
>   
> ambari-common/src/main/python/resource_management/libraries/providers/hdfs_resource.py
>  f1aa3e1 
>   
> ambari-common/src/main/python/resource_management/libraries/resources/hdfs_resource.py
>  5761fd6 
>   
> contrib/fast-hdfs-resource/src/main/java/org/apache/ambari/fast_hdfs_resource/Resource.java
>  da5a706 
>   
> contrib/fast-hdfs-resource/src/main/java/org/apache/ambari/fast_hdfs_resource/Runner.java
>  e4656c7 
> 
> Diff: https://reviews.apache.org/r/54874/diff/
> 
> 
> Testing
> -------
> 
> Manual testing with and without web hdfs.  Tested download files and 
> directories both when they existed locally and when they didn't.
> 
> 
> Thanks,
> 
> Tim Thorpe
> 
>

Reply via email to