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

chore!: replace rusoto with AWS SDK #2243

Merged
merged 1 commit into from
Mar 8, 2024

Conversation

mightyshazam
Copy link
Contributor

Description

This PR removes rusoto from the AWS crate while adding a dependency to the Rust AWS SDK

Related Issue(s)

#2210

Documentation

@github-actions github-actions bot added the binding/rust Issues for the Rust crate label Mar 4, 2024
Copy link

github-actions bot commented Mar 4, 2024

ACTION NEEDED

delta-rs follows the Conventional Commits specification for release automation.

The PR title and description are used as the merge commit message. Please update your PR title and description to match the specification.

@mightyshazam mightyshazam changed the title Remove rusoto chore: replace rusoto with AWS SDK Mar 4, 2024
@mightyshazam mightyshazam changed the title chore: replace rusoto with AWS SDK chore!: replace rusoto with AWS SDK Mar 4, 2024
@rtyler
Copy link
Member

rtyler commented Mar 4, 2024

Exciting! thanks for taking this on @mightyshazam , I'll take a more in depth look tomorrow during a commute 🚌

@mightyshazam
Copy link
Contributor Author

@rtyler Looks like I may have overlooked some elements. I was trying to do this without updating the logstorefactory trait to require async, but now I'm not seeing a good way around it.

rtyler
rtyler previously approved these changes Mar 7, 2024
Copy link
Member

@rtyler rtyler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🙏 nice work!

@rtyler rtyler enabled auto-merge (rebase) March 7, 2024 01:43
@rtyler rtyler disabled auto-merge March 7, 2024 01:43
@rtyler rtyler enabled auto-merge (rebase) March 7, 2024 01:44
auto-merge was automatically disabled March 7, 2024 01:59

Rebase failed

@rtyler rtyler enabled auto-merge (rebase) March 7, 2024 17:07
auto-merge was automatically disabled March 7, 2024 17:23

Rebase failed

@rtyler rtyler self-assigned this Mar 7, 2024
@rtyler
Copy link
Member

rtyler commented Mar 7, 2024

I will handle rebasing this, some of the rebase conflicts are due to other code which I helped bring in

@rtyler rtyler merged commit 2f53872 into delta-io:main Mar 8, 2024
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
binding/rust Issues for the Rust crate
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants