[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Inconsistency in S3 Transfer Operators #37213

Open
2 tasks done
jroachgolf84 opened this issue Feb 7, 2024 · 6 comments
Open
2 tasks done

Inconsistency in S3 Transfer Operators #37213

jroachgolf84 opened this issue Feb 7, 2024 · 6 comments
Assignees
Labels

Comments

@jroachgolf84
Copy link
Contributor

Description

Standardize similar AWS transfer operators to have similar functionality, as well as similar naming conventions.

This would start with updating transfer operators using S3 as a source or destination to have similar behavior and naming.

Use case/motivation

There seems to be substantial discrepancies between transfer operators, especially those using S3 as a source or destination. For example, in the S3ToSFTPOperator, only a single file can be transferred from the SFTP to the S3 bucket. However, in the S3ToFTPOperator an arbitrary number of files can be transferred from S3 to an FTP site. To add to this, the FTPToS3Operator only supports a single file being transferred.

These sort of discrepancies exist for a number of transfer operators. In addition to that, parameter names are quite different from operator to operator, which makes using them quite difficult.

This will allow for a more seamless experience for Airflow users, and ensure common functionality across similar tools.

Related issues

No related issues.

Are you willing to submit a PR?

  • Yes I am willing to submit a PR!

Code of Conduct

@jroachgolf84 jroachgolf84 added kind:feature Feature Requests needs-triage label for new issues that we didn't triage yet labels Feb 7, 2024
Copy link
boring-cyborg bot commented Feb 7, 2024

Thanks for opening your first issue here! Be sure to follow the issue template! If you are willing to raise PR to address this issue please do so, no need to wait for approval.

@Taragolis Taragolis added provider:amazon-aws AWS/Amazon - related issues area:providers good first issue and removed needs-triage label for new issues that we didn't triage yet labels Feb 7, 2024
@bricefotzo
Copy link

Hi, I would like to bé assigned on this as my first issue, if possible. @jroachgolf84 would it be ok for you?

@jroachgolf84
Copy link
Contributor Author

@bricefotzo, were you able to complete this issue? If not, I'm looking forward to working on it.

@bricefotzo
Copy link

@jroachgolf84 I have added those changes 3da0dcf

@eladkal
Copy link
Contributor
eladkal commented Jul 28, 2024

@bricefotzo do you plan to raise a PR for this issue?

@bricefotzo
Copy link

Yes, I'll create a PR this week

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants