bharos opened a new pull request, #13101: URL: https://github.com/apache/gravitino/pull/13101
### What changes were proposed in this pull request? Add `Google auth Credentials` and `Google auth HTTP` to the BSD section of `LICENSE.iceberg`, matching the entries already present in `LICENSE.bin`. ### Why are the changes needed? Since `531058d2e`, `iceberg/iceberg-common` declares `implementation(libs.google.auth.http)`, so `google-auth-library-oauth2-http` is bundled in the `gravitino-iceberg-rest-server` distribution. Its POM declares `google-auth-library-credentials` at compile scope, so that jar is bundled as well. Neither was listed in `LICENSE.iceberg`. No `NOTICE.iceberg` change is required — `NOTICE.bin` carries no google-auth entry either. Fix: #13100 ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? No functional change; `LICENSE.iceberg` only. Verified by inspection: 1. `gradle/libs.versions.toml` pins `google-auth = "1.28.0"`, and `iceberg/iceberg-common/build.gradle.kts` declares `implementation(libs.google.auth.http)`. 2. `google-auth-library-oauth2-http-1.28.0.pom` declares `google-auth-library-credentials` with no `<scope>`, i.e. compile scope, so both jars are bundled. 3. `LICENSE.bin` already lists both under the BSD section; this change makes `LICENSE.iceberg` consistent with it. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
