Skip to content

Build: Bump com.google.cloud:libraries-bom from 26.80.0 to 26.81.0#16382

Merged
huaxingao merged 1 commit into
apache:mainfrom
huaxingao:bump-gcp-libraries-bom-26.81.0
May 19, 2026
Merged

Build: Bump com.google.cloud:libraries-bom from 26.80.0 to 26.81.0#16382
huaxingao merged 1 commit into
apache:mainfrom
huaxingao:bump-gcp-libraries-bom-26.81.0

Conversation

@huaxingao
Copy link
Copy Markdown
Contributor

Build: Bump com.google.cloud:libraries-bom from 26.80.0 to 26.81.0

Supersedes #16377.
Co-authored-by: Cursor Agent cursoragent@cursor.com

Copy link
Copy Markdown
Contributor

@singhpk234 singhpk234 left a comment

Choose a reason for hiding this comment

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

There is a new dependency io.opentelemetry:opentelemetry-common:1.57.0 ideally we should be good, lets just double confirm from licence pov

io.opentelemetry:opentelemetry-sdk-trace:1.51.0
io.opentelemetry:opentelemetry-sdk:1.51.0
io.opentelemetry:opentelemetry-api:1.57.0
io.opentelemetry:opentelemetry-common:1.57.0
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

new dependency ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes — Apache 2.0, covered by the existing OpenTelemetry LICENSE entry. Thanks @ebyhr and @kevinjqliu for the license check!

Copy link
Copy Markdown
Member

@ebyhr ebyhr left a comment

Choose a reason for hiding this comment

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

I verified that the opentelemetry-common is licensed under the Apache License, which is the same as the licenses of other OpenTelemetry dependencies.

Copy link
Copy Markdown
Contributor

@kevinjqliu kevinjqliu left a comment

Choose a reason for hiding this comment

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

LGTM

the new dep opentelemetry-common references https://github.com/open-telemetry/opentelemetry-java in maven

It has apache license, and should be covered by the same OpenTelemetry LICENSE entry

iceberg/gcp-bundle/LICENSE

Lines 1536 to 1539 in fab0d10

This product bundles OpenTelemetry.
Project URL: https://opentelemetry.io/
License: Apache License, Version 2.0 - https://www.apache.org/licenses/LICENSE-2.0

it has no NOTICE file to propagate

@manuzhang
Copy link
Copy Markdown
Member

I also verified opentelemetry-common has no other dependencies bundled.

@huaxingao huaxingao merged commit 26a6f53 into apache:main May 19, 2026
37 checks passed
@huaxingao
Copy link
Copy Markdown
Contributor Author

Thanks everyone for the review!

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.

5 participants