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

Merged by avatar (May 16, 2025 5:27am UTC)

Merge details

  • Changes merged into main with 63203a26.
  • Deleted the source branch.

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