getsentry / getsentry/sentry-python

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

Ouverte
#3,095 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
Feature Integration: Clickhouse Python Spans
Langage dominant
Python
Étoiles
2.2k
Forks
669
Merge moyen
1 j 1 h
PR mergées (30 j)
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.

Guide de contribution

Ouvrir le guide de contribution

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.