getsentry / getsentry/sentry-java

Add SQL span data

オープン
#6,018 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る
Feature Java Platform: Java Spans
主要言語
Kotlin
スター
1.4k
フォーク
478
平均マージ
2日 23時間
マージ済み PR(30日)
67

説明

### Problem Statement

First-party JDBC and Android SQLite spans currently keep the SQL statement only in the span description and attach limited database metadata (`db.system` and `db.name`). They do not attach `db.query.text` or generate `db.query.summary`.

### Solution Brainstorm

Attach database query semantic data to first-party JDBC and Android SQLite spans:

- `db.query.text`: the parameterized or SDK-sanitized statement. We should run this through a sanitizer.
- `db.query.summary`: a low-cardinality operation/target summary such as `UPDATE users`.
- Consider attaching analyzer-derived operation, collection/table, and stored-procedure attributes where they match Sentry conventions.

Check sentry conventions for more (query derived) attributes we can attach.

There's currently no data collection option to gate sanitized/parameterized `db.query.text` or `db.query.summary` so we can just attach them. Best double check https://develop.sentry.dev/sdk/foundations/client/data-collection/ when implementing.

Closely related to https://github.com/getsentry/sentry-java/issues/6019

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

Start by locating the first-party JDBC and Android SQLite span instrumentation, then review Sentry database conventions and the linked issue 6019. Check the data-collection guidance before implementation. Done means the relevant spans attach sanitized or parameterized db.query.text and low-cardinality db.query.summary, with matching derived attributes where applicable.

索引モデルが issue の本文から書いたものです。

評価

技術スタック
android, java, kotlin, sqlite
領域
databases, observability
issue の種類
機能追加
難易度
4/5
見積もり時間
3〜5日
活発さ
活発
明瞭さ
おおむね明確
初心者へのやさしさ
48/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。