-
Notifications
You must be signed in to change notification settings - Fork 300
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
[#4661] improvement(iceberg-rest): Adding License and Notice to the i… #4909
base: main
Are you sure you want to change the base?
Conversation
… the iceberg-rest-server and gravitino server jars
@justinmclean can you help review this PR? Thanks! |
The iceberg LICENSE and NOTICE files are probably going to be different from the root gravitino one as the release artifact produced will have different contents, but this is obviously an improvement. |
Thanks @justinmclean for the feedback. Change to download LICENSE/NOTICE from iceberg repo and put them under META-INF. |
This is not quite right. The content of our LICENSE and NOTICE file would not be the same as the contents for the Apache Iceberg project; it will depend on what ends up in the release artifact. I can take a look and see what those contents should be if you want. |
Sure. Thanks @justinmclean. I reverted the last two commits. |
What changes were proposed in this pull request?
Adding license and notice to iceberg-rest-server and gravitino server jars
Why are the changes needed?
Fix: #4661 The release jar should contains LICENSE and NOTICE
Does this PR introduce any user-facing change?
No
How was this patch tested?
Build results attached.