Skip to content
Snippets Groups Projects

perf: Move publishing events to BigQuery and Amplitude to background

Merged Administrator requested to merge es+vb/background-publish-events into main

Created by: eseliger

On dotcom, we would do these on the request path. Valery noticed that these requests disproportionally often fail with 504 (timeout) errors. After checking some logs, we found that some of these requests actually take up to a minute and more. To our understanding, it is not critical that these happen on the request path, though. If they fail that would've been a problem before as well. Also, we now properly store away the local events all the time, which is a good improvement, I think.

Works on https://github.com/sourcegraph/sourcegraph/issues/27908

Merge request reports

Approval is optional

Merged by avatar (Apr 29, 2025 10:26am UTC)

Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading