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

Can the aws-sdk-cpp pin be loosened? #867

Closed
akrherz opened this issue Oct 3, 2022 · 4 comments · Fixed by #875
Closed

Can the aws-sdk-cpp pin be loosened? #867

akrherz opened this issue Oct 3, 2022 · 4 comments · Fixed by #875
Labels

Comments

@akrherz
Copy link

akrherz commented Oct 3, 2022

Comment:

I've been struggling with conda-forge dependency issues and now believe to have isolated it to the pinning on aws-sdk-cpp within this repo. Does this pin still need to exist with the current 6.0 branch version of 6.0.2, which should have ARROW-15141 fixed?

Thank you.

@h-vetinari
Copy link
Member

Thanks for the report!

Does this pin still need to exist with the current 6.0 branch version of 6.0.2, which should have ARROW-15141 fixed?

That issue was fixed precisely by adding the pin. The unsolved root source of the problem is: aws/aws-sdk-cpp#1809

The way I understood @pitrou, this is a hard blocker for removing the pin, and I'll treat it as such. Unfortunately, that means the only possibilities you have is to either convince aws-sdk-cpp to fix that issue or other consuming packages to expand their supported range of aws-sdk-cpp.

@akrherz
Copy link
Author

akrherz commented Oct 3, 2022

Thank you @h-vetinari, I see what you are saying and digging more I find awslabs/aws-c-io#515 merged and in a release, which references the sdk-cpp issue. Hmmm.

@h-vetinari
Copy link
Member

Thank you @h-vetinari, I see what you are saying and digging more I find awslabs/aws-c-io#515 merged and in a release, which references the sdk-cpp issue. Hmmm.

Good digging. Fix is unconfirmed AFAIU, but more importantly, not part of aws-sdk-cpp yet (which affects us less because we unbundle the dependencies; but it could mean that the sdk-code is not tested/compatible with newer aws-c-* deps). On top of that, the migration for the respective aws-c-io release is currently running into an unresolved problem.

@h-vetinari
Copy link
Member

This was done in #875 🥳

It's on an exploratory basis because we cannot test the previous failure (in absence of AWS infra); let's hope we don't have to revert it 🤞

@h-vetinari h-vetinari mentioned this issue Dec 4, 2022
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants