Skip to content

Add db.query.summary support to JDBC instrumentation #15994

Merged
laurit merged 3 commits intoopen-telemetry:mainfrom
trask:jdbc-summary
Jan 28, 2026
Merged

Add db.query.summary support to JDBC instrumentation #15994
laurit merged 3 commits intoopen-telemetry:mainfrom
trask:jdbc-summary

Conversation

@trask
Copy link
Copy Markdown
Member

@trask trask commented Jan 25, 2026

No description provided.

@trask trask force-pushed the jdbc-summary branch 15 times, most recently from 18ed64f to 2659301 Compare January 27, 2026 21:32
emitStableDatabaseSemconv()
? "BATCH INSERT jdbcunittest"
: "jdbcunittest")
span.hasName(emitStableDatabaseSemconv() ? "BATCH" : "jdbcunittest")
Copy link
Copy Markdown
Member Author

@trask trask Jan 27, 2026

Choose a reason for hiding this comment

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

summary is only BATCH because the individual summaries are different (inserting into different tables)

@trask trask force-pushed the jdbc-summary branch 2 times, most recently from 862a5f3 to 73d2302 Compare January 28, 2026 03:02
@trask trask marked this pull request as ready for review January 28, 2026 04:42
@trask trask requested a review from a team as a code owner January 28, 2026 04:42
@laurit laurit merged commit 18b477f into open-telemetry:main Jan 28, 2026
85 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