Skip to content

feat(aws-sdk): add RDS Data SQL telemetry#19259

Open
rsclarke wants to merge 2 commits into
open-telemetry:mainfrom
rsclarke:feat/aws-sdk-rds-data-sql
Open

feat(aws-sdk): add RDS Data SQL telemetry#19259
rsclarke wants to merge 2 commits into
open-telemetry:mainfrom
rsclarke:feat/aws-sdk-rds-data-sql

Conversation

@rsclarke

@rsclarke rsclarke commented Jul 17, 2026

Copy link
Copy Markdown

Adds SQL database semantic-convention enrichment to the existing AWS SDK v2 spans for RDS Data API ExecuteStatement and BatchExecuteStatement calls. This provides JDBC-aligned query sanitization, summaries, database attributes, and duration metrics without adding a generated RDS Data runtime dependency or creating a duplicate nested span.

  • Extract other_sql, database namespace, SQL text, parameterization, and parameter-set batch size through AWS SDK core APIs.
  • Preserve the existing RdsData.<operation> span name in legacy-only mode and use database query-summary names in stable database-semconv mode.
  • Keep deprecated ExecuteSql and transaction-control operations on the existing generic AWS RPC path.
  • Add isolated AWS SDK 2.5.54+ suites for library, javaagent, and library autoconfigure in legacy and stable database-semconv modes.
  • Exercise sync and async clients, sanitization, parameterized SQL, batch sizes 0/1/2, metrics, modeled errors, and generic transaction routing using only a local mock endpoint and fake credentials.
  • Add the RDS Data artifact to the reproducible latest-dependency pins.

Closes #19256

@linux-foundation-easycla

linux-foundation-easycla Bot commented Jul 17, 2026

Copy link
Copy Markdown

CLA Signed
The committers listed above are authorized under a signed CLA.

  • ✅ login: rsclarke / name: rsclarke (28ab2ee)

@rsclarke
rsclarke marked this pull request as ready for review July 17, 2026 09:08
@rsclarke
rsclarke requested a review from a team as a code owner July 17, 2026 09:08
Copilot AI review requested due to automatic review settings July 17, 2026 09:08

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds RDS Data API SQL telemetry to existing AWS SDK v2 spans without runtime coupling or duplicate spans.

Changes:

  • Enriches statement operations with sanitized SQL database attributes, naming, metrics, and errors.
  • Preserves generic routing for transaction operations.
  • Adds legacy/stable coverage across library, autoconfigure, and javaagent.

Reviewed changes

Copilot reviewed 15 out of 15 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
testing/.../AbstractAws2RdsDataTest.java Adds shared RDS Data telemetry tests.
testing/build.gradle.kts Adds the RDS Data test dependency.
library/.../Aws2RdsDataTest.java Configures library tests.
library/.../TracingExecutionInterceptor.java Routes RDS Data requests to the SQL instrumenter.
library/.../RdsDataSqlAttributesGetter.java Extracts SQL request metadata.
library/.../AwsSdkRequestType.java Defines the RDS Data request type.
library/.../AwsSdkRequest.java Classifies supported statement requests.
library/.../AwsSdkInstrumenterFactory.java Builds the database-enriched instrumenter.
library/.../AwsSdkTelemetry.java Wires the new instrumenter into telemetry.
library/build.gradle.kts Adds legacy and stable test suites.
library-autoconfigure/.../Aws2RdsDataTest.java Configures autoconfigure tests.
library-autoconfigure/build.gradle.kts Registers autoconfigure test suites.
javaagent/.../Aws2RdsDataTest.java Configures javaagent tests.
javaagent/build.gradle.kts Registers javaagent test suites.
.github/config/latest-dep-versions.json Pins the latest RDS Data dependency.

@opentelemetry-pr-dashboard

opentelemetry-pr-dashboard Bot commented Jul 17, 2026

Copy link
Copy Markdown

Pull request dashboard status

Status last refreshed: 2026-07-26 23:35:19 UTC.

  • Waiting on: Reviewers
  • Next step: Review the latest changes.

This automated status or its linked feedback items may be incorrect. If something looks wrong, please report it with the result you expected.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 15 out of 15 changed files in this pull request and generated no new comments.

rsclarke added 2 commits July 19, 2026 19:50
Enrich ExecuteStatement and BatchExecuteStatement spans with SQL database semantic conventions while preserving legacy AWS operation names. Add isolated local-mock coverage for library, javaagent, and autoconfigure modes.

Refs open-telemetry#19256
@rsclarke
rsclarke force-pushed the feat/aws-sdk-rds-data-sql branch from 72c6be5 to 2455b45 Compare July 19, 2026 11:59
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.

Add SQL database semantic conventions to AWS SDK v2 RDS Data API spans

2 participants