getsentry / getsentry/sentry-python

sqlalchemy integration does not track all databases

Closed
#6,573 6 comments 2 reactions 1 assignee Claimed by @ericapisani View on GitHub
Bug Python Spans Waiting for: Product Owner
Dominant language
Python
Stars
2.2k
Forks
669
Avg merge
1d 40m
Merged PRs (30d)
212

Description

### How do you use Sentry?

Sentry Saas (sentry.io)

### Version

2.61.1

### Steps to Reproduce

When using `snowflake-sqlalchemy` or `clickhouse-sqlalchemy` the recorded spans do not include the attribute `span.system`, which make it difficult to filter the queries.

The only span.system we get is `postgresql`, which in our case is also used through sqlalchemey (+asyncpg).

Is this a problem on the Sentry side, or is this something that should be fixed by the sqlalchemy extensions (snowflake and clickhouse)?

### Expected Result

No span.system is recorded for snowflake or clickhouse.

### Actual Result

All queries should include the right system.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.