You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spark.apache.org by vi...@apache.org on 2026/09/18 00:28:56 UTC
(spark-connect-gateway) branch main updated: [SPARK-59548] Upgrade the OpenTelemetry crates to 0.32 and tracing-opentelemetry to 0.33
This is an automated email from the ASF dual-hosted git repository.
viirya pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/spark-connect-gateway.git
The following commit(s) were added to refs/heads/main by this push:
new ef2d2ba [SPARK-59548] Upgrade the OpenTelemetry crates to 0.32 and tracing-opentelemetry to 0.33
ef2d2ba is described below
commit ef2d2ba63da1c2b0988425199d19c6f1affec134
Author: Liang-Chi Hsieh <vi...@gmail.com>
AuthorDate: Thu Sep 17 17:28:47 2026 -0700
[SPARK-59548] Upgrade the OpenTelemetry crates to 0.32 and tracing-opentelemetry to 0.33
### What changes were proposed in this pull request?
Move the seven `opentelemetry*` crates from 0.31 to 0.32, and
`tracing-opentelemetry` from 0.32 to 0.33:
```
opentelemetry 0.31 -> 0.32
opentelemetry_sdk 0.31 -> 0.32
opentelemetry-otlp 0.31 -> 0.32
opentelemetry-semantic-conventions 0.31 -> 0.32
opentelemetry-stdout 0.31 -> 0.32
opentelemetry-proto 0.31 -> 0.32
tracing-opentelemetry 0.32 -> 0.33
```
`Cargo.toml` and `Cargo.lock` only. No source changes were needed — the APIs this
repo uses are unchanged between 0.31 and 0.32.
This supersedes #8, which is closed.
### Why are the changes needed?
Dependabot opened #8 to bump `opentelemetry_sdk` on its own. That cannot compile:
the other six `opentelemetry*` crates stay at 0.31, so both 0.31 and 0.32 of
`opentelemetry` land in the dependency graph and the traits stop lining up.
Reproduced locally on `main`:
```
error[E0308]: mismatched types
--> crates/observability/src/tracing.rs:167:25
error[E0277]: the trait bound `opentelemetry_otlp::SpanExporter:
opentelemetry_sdk::trace::SpanExporter` is not satisfied
--> crates/observability/src/tracing.rs:199:30
error[E0277]: the trait bound `TraceContextPropagator: TextMapPropagator`
is not satisfied
--> crates/observability/src/tracing.rs:206:37
```
Moving the whole group to 0.32 is still not sufficient on its own, because
`tracing-opentelemetry` 0.32 itself depends on `opentelemetry` 0.31 and pulls it
back in:
```
error[E0277]: the trait bound `SdkTracer: opentelemetry::trace::tracer::Tracer`
is not satisfied
--> crates/observability/src/tracing.rs:209:65
```
0.33 is the `tracing-opentelemetry` release that tracks `opentelemetry` 0.32, so
all eight crates move at once.
### Does this PR introduce _any_ user-facing change?
No. Dependency versions only; no change to configuration, metrics, span names, or
any emitted telemetry.
### How was this patch tested?
The full CI quartet on this branch:
| check | result |
|---|---|
| `cargo test --workspace` | 204 passed, 0 failed, 10 ignored — same as the baseline on `main` |
| `cargo clippy --workspace --all-targets -- -D warnings` | clean |
| `cargo fmt --all --check` | clean |
| `cargo build --workspace` | clean |
The check that actually shows the version skew is gone is the lockfile resolving a
single `opentelemetry`:
```
$ grep -c '^name = "opentelemetry"' Cargo.lock
1
```
Beyond the quartet I ran the `#[ignore]`d integration tests, because the unit
tests are the weakest signal for this particular dependency group:
- `tracing_emits_spans_and_propagates_traceparent` passes. This one matters most:
feature unification in this group has previously caused application spans to be
dropped silently, and neither a build nor the unit tests would catch that.
- The Redis integration tests (`scg-store-redis`, `scg-ratelimit`) pass —
5 and 4 tests respectively — unaffected, as expected.
### Was this patch authored or co-authored using generative AI tooling?
Yes, co-authored with Claude Code.
Closes #27 from viirya/dep-01-opentelemetry-0.32.
Authored-by: Liang-Chi Hsieh <vi...@gmail.com>
Signed-off-by: Liang-Chi Hsieh <li...@databricks.com>
---
Cargo.lock | 41 +++++++++++++++++++++++++++--------------
Cargo.toml | 14 +++++++-------
2 files changed, 34 insertions(+), 21 deletions(-)
diff --git a/Cargo.lock b/Cargo.lock
index 3ea20ef..83d38ed 100644
--- a/Cargo.lock
+++ b/Cargo.lock
@@ -1894,9 +1894,9 @@ checksum = "7c87def4c32ab89d880effc9e097653c8da5d6ef28e6b539d313baaacfbafcbe"
[[package]]
name = "opentelemetry"
-version = "0.31.0"
+version = "0.32.0"
source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "b84bcd6ae87133e903af7ef497404dda70c60d0ea14895fc8a5e6722754fc2a0"
+checksum = "b0142c63252a9e054e68a4c61a5778f7b14f576274d593f8ce883d191a099682"
dependencies = [
"futures-core",
"futures-sink",
@@ -1908,9 +1908,9 @@ dependencies = [
[[package]]
name = "opentelemetry-otlp"
-version = "0.31.1"
+version = "0.32.0"
source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "1f69cd6acbb9af919df949cd1ec9e5e7fdc2ef15d234b6b795aaa525cc02f71f"
+checksum = "9966929966d17620d7c316c643ba62631826e10021409357772d5eea84f62c35"
dependencies = [
"http",
"opentelemetry",
@@ -1920,13 +1920,14 @@ dependencies = [
"thiserror 2.0.20",
"tokio",
"tonic",
+ "tonic-types",
]
[[package]]
name = "opentelemetry-proto"
-version = "0.31.0"
+version = "0.32.0"
source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "a7175df06de5eaee9909d4805a3d07e28bb752c34cab57fa9cff549da596b30f"
+checksum = "56d658ba1faf63f7b9c492cfbe6e0ec365440a16132d3270c1065f7b33f1b638"
dependencies = [
"opentelemetry",
"opentelemetry_sdk",
@@ -1937,15 +1938,15 @@ dependencies = [
[[package]]
name = "opentelemetry-semantic-conventions"
-version = "0.31.0"
+version = "0.32.1"
source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "e62e29dfe041afb8ed2a6c9737ab57db4907285d999ef8ad3a59092a36bdc846"
+checksum = "c913ac17a6c451661ee255f4625d143e51647ae78ebd969b75e41c4442f4fe47"
[[package]]
name = "opentelemetry-stdout"
-version = "0.31.0"
+version = "0.32.0"
source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "bc8887887e169414f637b18751487cce4e095be787d23fad13c454e2fb1b3811"
+checksum = "a1b1c6a247d79091f0062a5f4bd058589525cf987a8d4c169440d9c1be72f0ad"
dependencies = [
"chrono",
"opentelemetry",
@@ -1954,15 +1955,16 @@ dependencies = [
[[package]]
name = "opentelemetry_sdk"
-version = "0.31.0"
+version = "0.32.1"
source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "e14ae4f5991976fd48df6d843de219ca6d31b01daaab2dad5af2badeded372bd"
+checksum = "9b59f80e1ac4d5ff7a2db8fb6c80badb7f0f3f858211fba08dd9aaec750894f9"
dependencies = [
"futures-channel",
"futures-executor",
"futures-util",
"opentelemetry",
"percent-encoding",
+ "portable-atomic",
"rand 0.9.5",
"thiserror 2.0.20",
"tokio",
@@ -3615,6 +3617,17 @@ dependencies = [
"tonic-build",
]
+[[package]]
+name = "tonic-types"
+version = "0.14.6"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "73ab1b02061f83d519bba3caa167f88f261ef05720ab8ebc954ade70de3348e8"
+dependencies = [
+ "prost",
+ "prost-types",
+ "tonic",
+]
+
[[package]]
name = "tower"
version = "0.5.3"
@@ -3713,9 +3726,9 @@ dependencies = [
[[package]]
name = "tracing-opentelemetry"
-version = "0.32.1"
+version = "0.33.0"
source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "1ac28f2d093c6c477eaa76b23525478f38de514fa9aeb1285738d4b97a9552fc"
+checksum = "adbc64cba7137545b8044cb1fe9814f7aacf3c6b5f9b45be8bb5db538befdb26"
dependencies = [
"js-sys",
"opentelemetry",
diff --git a/Cargo.toml b/Cargo.toml
index 6806c53..2cdc0cd 100644
--- a/Cargo.toml
+++ b/Cargo.toml
@@ -61,13 +61,13 @@ prost-types = "0.14"
# Logging + telemetry
tracing = "0.1"
tracing-subscriber = { version = "0.3", features = ["env-filter", "json"] }
-opentelemetry = "0.31"
-opentelemetry_sdk = { version = "0.31", features = ["rt-tokio"] }
-opentelemetry-otlp = { version = "0.31", default-features = false, features = ["trace", "grpc-tonic", "tls-ring", "tls-webpki-roots"] }
-opentelemetry-semantic-conventions = "0.31"
-opentelemetry-stdout = "0.31"
-opentelemetry-proto = { version = "0.31", default-features = false, features = ["gen-tonic", "trace"] }
-tracing-opentelemetry = "0.32"
+opentelemetry = "0.32"
+opentelemetry_sdk = { version = "0.32", features = ["rt-tokio"] }
+opentelemetry-otlp = { version = "0.32", default-features = false, features = ["trace", "grpc-tonic", "tls-ring", "tls-webpki-roots"] }
+opentelemetry-semantic-conventions = "0.32"
+opentelemetry-stdout = "0.32"
+opentelemetry-proto = { version = "0.32", default-features = false, features = ["gen-tonic", "trace"] }
+tracing-opentelemetry = "0.33"
# Config + utilities
serde = { version = "1", features = ["derive"] }
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@spark.apache.org
For additional commands, e-mail: commits-help@spark.apache.org