feat(clickhouse_driver): Support span streaming

[alexander-alderman-webb]

Description

In the streaming path, use

• db.namespace instead of db.name
• db.system.name instead of db.system

Dropped attributes:

• db.params
• db.result
• query
• db.query_id

Tests for adding breadcrumbs based on spans have not been parametrized as breadcrumbs are not automatically created from spans with the new streaming lifecycle.

Adapting Tests

sed commands used for converting transaction context managers:

• sed -i '' 's/start_transaction(name="test_clickhouse_transaction") as transaction:/sentry_sdk.traces.start_span(name="custom parent") as span:/g'
• sed -i '' 's/start_transaction(name="test_clickhouse_transaction"):/sentry_sdk.traces.start_span(name="custom parent"):/g'

sed commands used for converting event capture:

• sed -i '' 's/capture_events,/capture_items,/g'
• sed -i '' 's/events = capture_events()/items = capture_items("event", "transaction", "span")/g'
• sed -i '' 's/event["spans"]/spans/g'

sed commands used for converting op:

• sed -i '' 's/"op"/"sentry.op"/g'

sed commands used for converting origin:

• sed -i '' 's/"origin"/"sentry.origin"/g'

sed commands used for converting description:

sed -i '' 's/"description"/"name"/g'

sed commands used for converting data to attributes:

• sed -i '' 's/"data"/"attributes"/g'

sed commands used for converting timestamps:

other test changes:

• sed -i '' 's/transaction_trace_id/trace_id/g'
• sed -i '' 's/transaction_span_id/span_id/g'
• sed -i '' 's/transaction.trace_id/span.trace_id/g'
• sed -i '' 's/transaction.span_id/span.span_id/g'
• sed -i '' '/same_process_as_parent/d'
• sed -i '' '/db.result/d'
• sed -i '' '/db.params/d'
• sed -i '' 's/"timestamp"/"end_timestamp"/g'
• sed -i '' 's/"db.system"/"db.system.name"/g'
• sed -i '' 's/"db.name"/"db.namespace"/g'

Issues

Closes #6011

Reminders

• Please add tests to validate your changes, and lint your code using uv run ruff.
• Add GH Issue ID & Linear ID (if applicable)
• PR title should use conventional commit style (feat:, fix:, ref:, meta:)
• For external contributors: CONTRIBUTING.md, Sentry SDK development docs, Discord community

[github-actions]

Codecov Results 📊

✅ 88791 passed | ⏭️ 6025 skipped | Total: 94816 | Pass Rate: 93.65% | Execution Time: 291m 24s

📊 Comparison with Base Branch

Metric	Change
Total Tests	📈 +15
Passed Tests	📈 +15
Failed Tests	—
Skipped Tests	—

All tests are passing successfully.

✅ Patch coverage is 91.89%. Project has 2357 uncovered lines.
✅ Project coverage is 89.9%. Comparing base (base) to head (head).

Files with missing lines (1)

File	Patch %	Lines
sentry_sdk/integrations/clickhouse_driver.py	91.89%	⚠️ 3 Missing and 3 partials

Coverage diff

@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    89.90%    89.90%        —%
==========================================
  Files          192       192         —
  Lines        23321     23340       +19
  Branches      8020      8030       +10
==========================================
+ Hits         20967     20983       +16
- Misses        2354      2357        +3
- Partials      1326      1328        +2

---

Generated by Codecov Action

[sentrivana]

See comments, feels like we could migrate more data to attributes?

[sentrivana]

IIRC the decorator also wraps the decorated func in functools.wraps(), so we should add that here manually

[alexander-alderman-webb]

yes, a few other integrations are also affected so I'll go ahead and update them as well. 30ffaad

[sentrivana]

This could be https://getsentry.github.io/sentry-conventions/attributes/db/#db-query-text?

[alexander-alderman-webb]

db.query.text should have placeholders according to the description, and the query passed to Connection.send_query already contains substituted values.

You can observe this in the tests for the span descriptions (there are no assertions on the query attribute):

sentry-python/tests/integrations/clickhouse_driver/test_clickhouse_driver.py

Line 326 in afff0e2

"description": "SELECT sum(x) FROM test WHERE x > 150",

[sentrivana]

https://getsentry.github.io/sentry-conventions/attributes/db/#db-query-parameter-key?

[alexander-alderman-webb]

We didn't add the attribute in SQLAlchemy because it was unclear how parameters for batched queries are reported. The same applies to this driver.