getsentry / getsentry/sentry-python

Make ClickHouse integration work with `django-clickhouse-backend`

Offen
#3,095 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Feature Integration: Clickhouse Python Spans
Vorherrschende Sprache
Python
Sterne
2.2k
Forks
669
Ø Merge
1 T. 1 Std.
Gemergte PRs (30 T.)
213

Beschreibung

### 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.

Beitragsleitfaden

Beitragsleitfaden öffnen

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.