Skip to content

Conversation

DemetrisChr
Copy link
Contributor

@DemetrisChr DemetrisChr commented Oct 8, 2025

Motivation

The Observability RFC expects having separate dispatch_to_server spans. Currently, we are recording all dispatch-level attributes on the top-level operation span.

Changes

  • Add dispatch spans for KV operations, one span per server round-trip. Update the dispatch-level attribute names so they conform to the latest semantic conventions
  • Update the threshold logging tracer so it can handle both the legacy top level spans that have dispatch-level attributes (temporarily, for compatibility with HTTP operations) and the new dispatch spans.
  • Add "canonical" address & port fields to memd sessions, these are the default (non-external) hostname & port as they appear in the config, which are now being reported in the new server.endpoint and server.port dispatch span attributes.

@DemetrisChr DemetrisChr force-pushed the CXXCBC-736-kv-dispatch-spans branch from b1db869 to 89c6aa8 Compare October 8, 2025 13:35
@DemetrisChr DemetrisChr force-pushed the CXXCBC-736-kv-dispatch-spans branch from 89c6aa8 to 8f269c9 Compare October 8, 2025 13:36
@DemetrisChr DemetrisChr merged commit ad4836b into couchbase:main Oct 8, 2025
64 of 74 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.

2 participants