[jira] [Commented] (AIRFLOW-2999) Add S3DownloadOperator

["lovk korm (Jira)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/AIRFLOW-2999?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17085644#comment-17085644 ] 

lovk korm commented on AIRFLOW-2999:
------------------------------------

[~feluelle] 
there is download_file function in s3hook probably it just need to wrap it with a new operator called S3DownloadOperator 

> Add S3DownloadOperator
> ----------------------
>
>                 Key: AIRFLOW-2999
>                 URL: https://issues.apache.org/jira/browse/AIRFLOW-2999
>             Project: Apache Airflow
>          Issue Type: Task
>    Affects Versions: 1.10.0
>            Reporter: jack
>            Priority: Major
>
> The [S3_hook |https://github.com/apache/incubator-airflow/blob/master/airflow/hooks/S3_hook.py#L177] has get_key method that returns boto3.s3.Object it also has load_file method which loads file from local file system to S3.
>  
> What it doesn't have is a method to download a file from S3 to the local file system.
> Basicly it should be something very simple... an extention to the get_key method with parameter to the destination on local file system adding a code for taking the boto3.s3.Object and save it on the disk.  Note: that it can be more than 1 file if the user choose a folder in S3.
>  
> +*Update:*+
> As discussed in comments instead having the property in the hook it's better to mirror the GoogleCloudStorageDownloadOperator and have S3DownloadOperator
>  
>  
>  



--
This message was sent by Atlassian Jira
(v8.3.4#803005)