----------------------------------------------------------- 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 > >