feat(gax): link T3 attempt spans to parent context and clean up on operation completion by jinseopkim0 · Pull Request #14459 · googleapis/google-cloud-java

feat: add T3 client request operation spans to OpenTelemetryTracingTracer

jinseopkim0

committed

f5ae43d

test: verify operationFailed ends active attempt span

jinseopkim0

committed

f89860e

fix: record error attributes on attempt span when operation fails

jinseopkim0

committed

e81fd4f

fix: use operation as fallback name when attemptSpanName is attempt

jinseopkim0

committed

9987d8a

fix: guard empty operation span name and ensure operationSpan ended in finally

jinseopkim0

committed

c769a57

refactor: derive operationSpanName directly from attemptSpanName

jinseopkim0

committed

8980aa3

fix: mark spans volatile and append /attempt to attempt span name

jinseopkim0

committed

b48b0e9

fix: use local variables for volatile span fields to prevent race conditions

jinseopkim0

committed

8baed77

fix: use Strings.isNullOrEmpty for fullMethodName check

jinseopkim0

committed

46fbef2

test: avoid fully qualified Span and SpanContext class names

jinseopkim0

committed

d687b54

fix: clear operationSpan immediately to prevent concurrent double-end

jinseopkim0

committed

d3153b0

fix: copy operationSpan to local variable in attemptStarted

jinseopkim0

committed

03dd4b3

fix: capture operationContext at creation and use explicit lock for thread safety

jinseopkim0

committed

7068ce7

fix: set StatusCode.ERROR on failed and cancelled operation spans

jinseopkim0

committed

6355772

fix: return early if operationSpan is null in attemptStarted to prevent span leaks

jinseopkim0

committed

942f8c2

refactor: end attempt spans outside lock to reduce lock contention

jinseopkim0

committed

64ac55a

fix: suppress MustBeClosedChecker on OpenTelemetry inScope

jinseopkim0

committed

dd7b25f

fix(gax): link attempt spans to parent context and clean up on operation completion

jinseopkim0

committed

32187cf

fix(gax): release lock before ending previous attemptSpan in attemptStarted

jinseopkim0

committed

191913c

Navigation Menu

feat(gax): link T3 attempt spans to parent context and clean up on operation completion - #14459

feat(gax): link T3 attempt spans to parent context and clean up on operation completion#14459
jinseopkim0 wants to merge 19 commits into
mainfrom
feat-t3-client-request-spans

Commits on Sep 21, 2026

feat: add T3 client request operation spans to OpenTelemetryTracingTracer

test: verify operationFailed ends active attempt span

fix: record error attributes on attempt span when operation fails

fix: use operation as fallback name when attemptSpanName is attempt

fix: guard empty operation span name and ensure operationSpan ended in finally

refactor: derive operationSpanName directly from attemptSpanName

fix: mark spans volatile and append /attempt to attempt span name

fix: use local variables for volatile span fields to prevent race conditions

fix: use Strings.isNullOrEmpty for fullMethodName check

test: avoid fully qualified Span and SpanContext class names

fix: clear operationSpan immediately to prevent concurrent double-end

fix: copy operationSpan to local variable in attemptStarted

fix: capture operationContext at creation and use explicit lock for thread safety

fix: set StatusCode.ERROR on failed and cancelled operation spans

fix: return early if operationSpan is null in attemptStarted to prevent span leaks

refactor: end attempt spans outside lock to reduce lock contention

fix: suppress MustBeClosedChecker on OpenTelemetry inScope

fix(gax): link attempt spans to parent context and clean up on operation completion

fix(gax): release lock before ending previous attemptSpan in attemptStarted