You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@iceberg.apache.org by "amogh-jahagirdar (via GitHub)" <gi...@apache.org> on 2026/05/07 16:34:34 UTC
[PR] GCP: Backport License/Notice fix from #16182 to 1.10.x, excluding woodstox/gcs-analytics and JSR [iceberg]
amogh-jahagirdar opened a new pull request, #16244:
URL: https://github.com/apache/iceberg/pull/16244
This change does two things:
1. Copied over the License/Notice for the azure bundle as of #16181 via
```
git checkout e4028bf6a012691933441387a893eb921a780438 -- gcp-bundle/LICENSE gcp-bundle/NOTICE
```
2. After generating runtime deps https://gist.github.com/amogh-jahagirdar/fc7e2caf02cd343706d6671f5e3721d4
compared with main's runtime-deps and saw the following differences.
a.) Main bundles gcs analytics while 1.10 does not
b.) Main bundles woodstox while 1.10 does not
c.) Main does not bundle JSR while 1.10 does. For this since there's LGPL concerns with JSR and it's only needed at compile time, I just went ahead and excluded it.
I went back and excluded the license entries for gcs analytics/woodstox.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] GCP: Backport License/Notice fix from #16182 to 1.10.x, excluding woodstox/gcs-analytics and JSR [iceberg]
Posted by "amogh-jahagirdar (via GitHub)" <gi...@apache.org>.
amogh-jahagirdar commented on code in PR #16244:
URL: https://github.com/apache/iceberg/pull/16244#discussion_r3203095430
##########
gcp-bundle/build.gradle:
##########
@@ -23,6 +23,13 @@ project(":iceberg-gcp-bundle") {
tasks.jar.dependsOn tasks.shadowJar
+ configurations {
+ implementation {
+ exclude group: 'com.google.code.findbugs', module: 'jsr305'
Review Comment:
Normally, I wouldn't do this in a patch release but I think there's a real concern with jsr305 and LGPL licnesing. Considering it's compile time only, I just excluded it along with sl4j (but slf4j was already excluded through a different mechanism)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] GCP: Backport License/Notice fix from #16182 to 1.10.x, excluding woodstox/gcs-analytics and JSR [iceberg]
Posted by "kevinjqliu (via GitHub)" <gi...@apache.org>.
kevinjqliu commented on code in PR #16244:
URL: https://github.com/apache/iceberg/pull/16244#discussion_r3203443736
##########
gcp-bundle/build.gradle:
##########
@@ -23,6 +23,13 @@ project(":iceberg-gcp-bundle") {
tasks.jar.dependsOn tasks.shadowJar
+ configurations {
+ implementation {
+ exclude group: 'com.google.code.findbugs', module: 'jsr305'
Review Comment:
nice! agreed, this is a backport of https://github.com/apache/iceberg/pull/16106/
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] GCP: Backport License/Notice fix from #16182 to 1.10.x, excluding woodstox/gcs-analytics and JSR [iceberg]
Posted by "amogh-jahagirdar (via GitHub)" <gi...@apache.org>.
amogh-jahagirdar merged PR #16244:
URL: https://github.com/apache/iceberg/pull/16244
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org