spring-cloud-aws icon indicating copy to clipboard operation
spring-cloud-aws copied to clipboard

Published 20 hours ago • verified publisher icon spring-attic

Support AWS SDK 2.0

Open mv78 opened this issue 8 years ago • 21 comments

AWS currently in the process or releasing SDK 2.0. It would be good to start integrating it into the project as it progresses to GA.

mv78 avatar Oct 06 '17 23:10 mv78

@mv78 just checked today. At least from the information I found they are in the preview. Hence we are waiting once AWS has released a final 2.0. I have updated the project to use the latest AWS SDK 1.x

aemruli avatar Dec 19 '17 17:12 aemruli

Can we upgrade AWS SDK v1.11.321 (latest as of May, 2018) ..? @aemruli

Or please share your last commit, i will do similar.

ajainy avatar May 16 '18 19:05 ajainy

@ajainy we have upgraded master to 1.11.336

aemruli avatar Jun 13 '18 13:06 aemruli

Are there any news on this? Usage with JDK11 gives some warnings on a class used in the old V1 of aws-sdk-java

WARNING: An illegal reflective access operation has occurred WARNING: Illegal reflective access by com.amazonaws.util.XpathUtils (file:/Users/michael/.m2/repository/com/amazonaws/aws-java-sdk-core/1.11.415/aws-java-sdk-core-1.11.415.jar) to method com.sun.org.apache.xpath.internal.XPathContext.getDTMManager() WARNING: Please consider reporting this to the maintainers of com.amazonaws.util.XpathUtils WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations WARNING: All illegal access operations will be denied in a future release

michl-b avatar May 02 '19 07:05 michl-b

Hi guys

Any news about AWS SDK version 2?

kmualem avatar May 23 '19 18:05 kmualem

Yes, problem with AWS SDK version 1 is it doesn't play nice with JDK 9 as it duplicates packages across individual jars.

Although an upgrade like this would touch a lot of classes it wouldn't be too hard to do as it's a simple job to upgrade to the new SDK and the new SDK is a lot easier to work with (imo it has a more fluent api).

michaelwiles avatar Jul 13 '19 19:07 michaelwiles

See another reason to do this: https://github.com/spring-cloud/spring-cloud-aws/issues/491 Also here: https://github.com/spring-projects/spring-integration-aws/issues/155

Thanks

artembilan avatar Sep 13 '19 17:09 artembilan

I started working on this here https://github.com/tinexw/spring-cloud-aws/tree/gh-267

If anyone wants to join, please let me know!

I would also like to get some insight from the maintainers what the best way to go about this is. I just started fixing compile errors and committed everything in one big commit. Not sure if that is a good idea.

Also, I'm just assuming we don't need to care about backwards compatibility. But is that correct?

tinexw avatar Oct 04 '19 00:10 tinexw

@tinexw not sure how this would work as updating major versions of a dependency would be a breaking change

spencergibb avatar Oct 09 '19 02:10 spencergibb

Yeah, that's why I was asking what the preferred approach in this case is. Release a new major version?

tinexw avatar Oct 09 '19 07:10 tinexw

We would be very happy to take this upgrade PR, but we can't put backwards-incompatible changes within the Hoxton release train, cause it's not going to be a major. We will be shortly deciding if our next release train will be major (we will probably branch it out in a few months). If yes, then these changes could be merged there.

OlgaMaciaszek avatar Nov 18 '19 16:11 OlgaMaciaszek

Ok, thanks for the update. There is quite a lot left. I will pick it up again once you know for sure if it’s going to be a major release.

tinexw avatar Nov 18 '19 17:11 tinexw

Yet another reason to upgrade: EKS supports 2.10.11+ only... Any ETA yet?

sp00m avatar Jan 22 '20 16:01 sp00m

Is this still being worked on? The 2.0 SDK would be really nice to have.

tcheek-bw avatar Feb 05 '20 16:02 tcheek-bw

@tinexw started working on it and now that master would be a major release we would be happy to accept it.

spencergibb avatar Feb 05 '20 16:02 spencergibb

Good to know :-) I will pick it back up. Would be great to get some support especially in regards to testing later on.

tinexw avatar Feb 05 '20 16:02 tinexw

I created a PR with my changes to far: https://github.com/spring-cloud/spring-cloud-aws/pull/524 (Original comment moved to PR as well.)

tinexw avatar Feb 10 '20 12:02 tinexw

Just as an FYI, this ticket blocked us going forward, so we decided to develop our own abstraction for our Spring WebFlux projects, so that we could connect to SQS from within EKS: https://github.com/qudini/qudini-reactive/tree/master/qudini-reactive-sqs. Very SQS (and WebFlux) focused obviously, and by no means intended to replace https://github.com/spring-cloud/spring-cloud-aws, but maybe it can help resolving this issue somehow :)

sp00m avatar Jul 13 '20 17:07 sp00m

Thanks @sp00m! We are going to move to SDK v2 in 3.0 and revisit SQS integration so we are definitely going to look at your repository for an inspiration.

maciejwalkowiak avatar Jul 13 '20 22:07 maciejwalkowiak

Perhaps Spring Cloud AWS 3.0 could just be an upgrade to SDK v2 in order to knock that out sooner rather than later?

We're starting to hit issues like SSO that simply aren't supported in SDK v1.

internetstaff avatar Apr 12 '21 19:04 internetstaff

The repo that would handle this is https://github.com/awspring/spring-cloud-aws

spencergibb avatar Apr 12 '21 20:04 spencergibb