Re: [PR] Add http to s3 operator [airflow]

2023-10-26 Thread via GitHub
potiuk merged PR #35176: URL: https://github.com/apache/airflow/pull/35176 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@airflow.a

Re: [PR] Add http to s3 operator [airflow]

2023-10-25 Thread via GitHub
vincbeck commented on code in PR #35176: URL: https://github.com/apache/airflow/pull/35176#discussion_r1372312570 ## airflow/providers/amazon/aws/transfers/http_to_s3.py: ## @@ -0,0 +1,163 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor

Re: [PR] Add http to s3 operator [airflow]

2023-10-25 Thread via GitHub
simonprydden commented on code in PR #35176: URL: https://github.com/apache/airflow/pull/35176#discussion_r1372280177 ## airflow/providers/amazon/aws/transfers/http_to_s3.py: ## @@ -0,0 +1,163 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more contrib

Re: [PR] Add http to s3 operator [airflow]

2023-10-25 Thread via GitHub
vincbeck commented on code in PR #35176: URL: https://github.com/apache/airflow/pull/35176#discussion_r1372268503 ## tests/system/providers/amazon/aws/example_http_to_s3.py: ## @@ -0,0 +1,83 @@ +# Licensed to the Apache Software Foundation (ASF) under one Review Comment: Gla

Re: [PR] Add http to s3 operator [airflow]

2023-10-25 Thread via GitHub
simonprydden commented on code in PR #35176: URL: https://github.com/apache/airflow/pull/35176#discussion_r1372265659 ## tests/system/providers/amazon/aws/example_http_to_s3.py: ## @@ -0,0 +1,83 @@ +# Licensed to the Apache Software Foundation (ASF) under one Review Comment:

Re: [PR] Add http to s3 operator [airflow]

2023-10-25 Thread via GitHub
vincbeck commented on PR #35176: URL: https://github.com/apache/airflow/pull/35176#issuecomment-1779949120 I guess it is safe to put it in review (unless you have a good reason) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

Re: [PR] Add http to s3 operator [airflow]

2023-10-25 Thread via GitHub
vincbeck commented on code in PR #35176: URL: https://github.com/apache/airflow/pull/35176#discussion_r1372253036 ## airflow/providers/amazon/aws/transfers/http_to_s3.py: ## @@ -0,0 +1,163 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor

Re: [PR] Add http to s3 operator [airflow]

2023-10-25 Thread via GitHub
Taragolis commented on code in PR #35176: URL: https://github.com/apache/airflow/pull/35176#discussion_r1372244799 ## docs/apache-airflow-providers-amazon/transfer/http_to_s3.rst: ## @@ -0,0 +1,53 @@ + .. Licensed to the Apache Software Foundation (ASF) under one +or more co

Re: [PR] Add http to s3 operator [airflow]

2023-10-25 Thread via GitHub
simonprydden commented on code in PR #35176: URL: https://github.com/apache/airflow/pull/35176#discussion_r1372198163 ## docs/apache-airflow-providers-amazon/transfer/http_to_s3.rst: ## @@ -0,0 +1,53 @@ + .. Licensed to the Apache Software Foundation (ASF) under one +or more

Re: [PR] Add http to s3 operator [airflow]

2023-10-25 Thread via GitHub
Taragolis commented on code in PR #35176: URL: https://github.com/apache/airflow/pull/35176#discussion_r1372191952 ## docs/apache-airflow-providers-amazon/transfer/http_to_s3.rst: ## @@ -0,0 +1,53 @@ + .. Licensed to the Apache Software Foundation (ASF) under one +or more co

[PR] Add http to s3 operator [airflow]

2023-10-25 Thread via GitHub
simonprydden opened a new pull request, #35176: URL: https://github.com/apache/airflow/pull/35176 Adding HttpToS3Operator and unit tests. --- **^ Add meaningful description above** Read the **[Pull Request Guidelines](https://github.com/apache/airflow