getsentry / getsentry/sentry-python
Make ClickHouse integration work with `django-clickhouse-backend`
- Dominant language
- Python
- Stars
- 2.2k
- Forks
- 669
- Avg merge
- 1d 1h
- Merged PRs (30d)
- 213
Description
### Problem Statement
The ClickHouse integration doesn't create spans if used in Django via `django-clickhouse-backend`.
This first surfaced in https://github.com/getsentry/sentry-python/issues/3088 where the whole integration would error out because `_sentry_span` was not set on the connection as expected. We're [patching the integration](https://github.com/getsentry/sentry-python/pull/3096) to not do anything if `_sentry_span` is not there, but it'd be good to have proper support for `django-clickhouse-backend`.
### Solution Brainstorm
Looks like the spans are not started the expected way (`clickhouse_driver.connection.Connection.send_query` is not called?), so we might need to wrap an additional function that `django-clickhouse-backend` uses to start queries.
Contributor guide
Assessment
This issue has not been assessed yet.