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

Remove ion-java #2969

Closed
wants to merge 1 commit into from
Closed

Remove ion-java #2969

wants to merge 1 commit into from

Conversation

jobarr-amzn
Copy link

Description of changes: Java SDK v2 dropped support for Ion, and it’s unsupported for v1. The version of ion-java (software.amazon.ion:ion-java:1.0.2) bundled with aws-sdk-java is substantially behind the current version (com.amazon.ion:ion-java:1.9.6) and uses a different java package. This causes customers using aws-sdk-java to get an out-dated version in their dependencies. It would be better to have an up-to-date version or no transitive dependency so that customers don't miss out on the bug fixes and performance improvements ion-java has made since then.

This commit removes ion-java from aws-sdk-java, removing also all the code that depends on it.
If it is preferable, I also have a patch ready which upgrades the ion-java version instead.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Java SDK v2 dropped support for Ion, and it’s unsupported for v1.
The version of ion-java (software.amazon.ion:ion-java:1.0.2) bundled with aws-sdk-java is substantially behind the current version (com.amazon.ion:ion-java:1.9.6) and uses a different java package.
This causes customers using aws-sdk-java to get an out-dated version in their dependencies. It would be better to have an up-to-date version or no transitive dependency so that customers don't miss out on the bug fixes and  performance improvements ion-java has made since then.

This commit removes ion-java from aws-sdk-java, removing also all the code that depends on it.
@zdenda-online
Copy link

Thanks, I think removal is a better option if it is not supported.

Any ETA to merge and release a new version? (whoever is in charge 😁 )

@jumraiya
Copy link

+1 on getting this merged.

@davidh44
Copy link
Contributor

Ion has been removed from Java v1

@davidh44 davidh44 closed this Jan 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants