Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

revert: "chore(clickhouse): Capture final SQL in Sentry errors" #21479

Merged
merged 1 commit into from
Apr 10, 2024

Conversation

pauldambra
Copy link
Member

Reverts #21436

see https://posthog.slack.com/archives/C0185UNBSJZ/p1712779189666039

We've seen multiple thousands of errors in Celery that appear to start when this PR deployed

@timgl timgl merged commit 1562781 into master Apr 10, 2024
71 of 72 checks passed
@timgl timgl deleted the revert-21436-capture-full-error-sql branch April 10, 2024 21:13
Twixes added a commit that referenced this pull request Apr 19, 2024
Twixes added a commit that referenced this pull request Apr 22, 2024
* Revert "revert: "chore(clickhouse): Capture final SQL in Sentry errors" (#21479)"

This reverts commit 1562781.

* Only upgrade `clickhouse-driver` to 0.2.6

* Add "Run migrations for this PR"
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants