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

Core, Spark, Flink, Hive: Remove unused failsafe dependency from core and add Failsafe to runtime LICENSE(s) #11816

Merged

Conversation

amogh-jahagirdar
Copy link
Contributor

@amogh-jahagirdar amogh-jahagirdar commented Dec 18, 2024

This change removes unused failsafe from core (it was an oversight to include it in core as part of c0d73f4#diff-49a96e7eea8a94af862798a45174e6ac43eb4f8b4bd40759b5da63ba31ec3ef7 , we ended up isolating the retry behavior just to AWS).

For clarity this change also adds Failsafe to the Runtime LICENSE(s) for the runtime artifacts which depend on iceberg-aws just to fall in line with our practice of including dependencies (although it's not required as per https://infra.apache.org/licensing-howto.html#alv2-dep since Failsafe is Apache 2.0 Licensed).

Note Failsafe has no Notice of its own.

Comment on lines +725 to +728
Group: dev.failsafe Name: failsafe Version: 3.3.2
Project URL (from POM): https://github.com/failsafe-lib/failsafe
License (from POM): Apache License, Version 2.0 - https://www.apache.org/licenses/LICENSE-2.0.txt

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I just followed the KC format for the license, cc @bryanck

@amogh-jahagirdar
Copy link
Contributor Author

Thanks for the reviews @nastra @Fokko!

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

Successfully merging this pull request may close these issues.

3 participants