Skip to content

Mongo dbo demo - #281

Draft
xuan-cao-swi wants to merge 6 commits into
mainfrom
mongo-dbo-demo
Draft

Mongo dbo demo#281
xuan-cao-swi wants to merge 6 commits into
mainfrom
mongo-dbo-demo

Conversation

@xuan-cao-swi

@xuan-cao-swi xuan-cao-swi commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Description

  • Simplified annotate_span_and_sql flow
  • Supports multiple MongoDB versions:
    • Mongo < 2.23.0: Patches Mongo::Server::ConnectionBase.deliver
    • Mongo >= 2.23.0: Patches Mongo::Tracing::OpenTelemetry::OperationTracer.execute_with_span

Test (if applicable)

Comment thread lib/solarwinds_apm/patch/tag_sql/sw_mongo_patch.rb Fixed
Comment thread lib/solarwinds_apm/patch/tag_sql/sw_mongo_patch.rb Fixed
Comment thread lib/solarwinds_apm/patch/tag_sql/sw_mongo_patch.rb Fixed
# ConnectionBase#deliver exists before 2.22.0 and remains the common
# serialization path in 2.23.0 and newer. Do not couple this patch to
# either of the driver's OpenTelemetry integration implementations.
if defined?(::Mongo::Server::ConnectionBase)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants