Skip to content

enhance: Add trace context snapshot to OpContext#93

Merged
yhmo merged 4 commits into
zilliztech:masterfrom
congqixia:enhance/opctx_trace
Jun 16, 2026
Merged

enhance: Add trace context snapshot to OpContext#93
yhmo merged 4 commits into
zilliztech:masterfrom
congqixia:enhance/opctx_trace

Conversation

@congqixia

Copy link
Copy Markdown
Contributor

Store an owned trace snapshot in OpContext so cache-layer code can propagate tracing metadata without depending on borrowed TraceContext buffers. Expose helpers to set trace data and create temporary TraceContext views for the existing tracer API.

Keep async warmup isolated from caller tracing by constructing a fresh OpContext with only the cancellation token. Add tracer and cache slot tests to lock in the snapshot lifetime and warmup propagation semantics.

Store an owned trace snapshot in OpContext so cache-layer code can
propagate tracing metadata without depending on borrowed TraceContext
buffers. Expose helpers to set trace data and create temporary
TraceContext views for the existing tracer API.

Keep async warmup isolated from caller tracing by constructing a fresh
OpContext with only the cancellation token. Add tracer and cache slot
tests to lock in the snapshot lifetime and warmup propagation
semantics.

Signed-off-by: Congqi Xia <congqi.xia@zilliz.com>
Signed-off-by: Congqi Xia <congqi.xia@zilliz.com>
Comment thread include/common/Tracer.h
Comment thread include/common/OpContext.h Outdated
Comment thread include/common/OpContext.h
Signed-off-by: Congqi Xia <congqi.xia@zilliz.com>
Comment thread include/common/Tracer.h
Comment thread include/common/Tracer.h
Signed-off-by: Congqi Xia <congqi.xia@zilliz.com>

@liliu-z liliu-z left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

/lgtm
/approve

@yhmo yhmo merged commit edbb0a7 into zilliztech:master Jun 16, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants