Rewrite precise-code-intel-worker in Go
Created by: efritz
This is a rewrite of the precise-code-intel-worker in Go. This does not change the current entrypoints as we're not ready to switch to the new code yet. I am planning on manually testing requests against the old and new servers to ensure the behavior is the same during usage. A follow-up PR(s) will remove the old code and switch to the new code in build and deployment environments.
This new code needs to emit metric, add tracing to requests, and wrap errors properly before being considered ready for production.
Opening PR now for discussion as it's a large chunk of code. This effort is tracked in https://github.com/sourcegraph/sourcegraph/issues/9965.
Merge request reports
Activity
Created by: codecov[bot]
Codecov Report
Merging #10105 into master will not change coverage. The diff coverage is
69.03%
.@@ Coverage Diff @@ ## master #10105 +/- ## ======================================= Coverage 43.85% 43.85% ======================================= Files 1419 1419 Lines 78003 78003 Branches 6904 6979 +75 ======================================= Hits 34210 34210 Misses 40591 40591 Partials 3202 3202
Flag Coverage Δ #go 47.02% <69.03%> (ø)
#typescript 34.19% <ø> (ø)
#unit 43.85% <69.03%> (ø)
Impacted Files Coverage Δ ...e-code-intel-worker/internal/correlation/errors.go 0.00% <0.00%> (ø)
...tel-worker/internal/correlation/lsif/result_set.go 0.00% <0.00%> (ø)
...-code-intel-worker/internal/existence/existence.go 0.00% <0.00%> (ø)
...de-intel-worker/internal/correlation/lsif/range.go 26.31% <26.31%> (ø)
...-intel-worker/internal/correlation/lsif/moniker.go 27.27% <27.27%> (ø)
...recise-code-intel-worker/internal/worker/worker.go 41.88% <41.88%> (ø)
...ode-intel-worker/internal/correlation/correlate.go 61.42% <61.42%> (ø)
...intel-worker/internal/correlation/lsif/document.go 71.42% <71.42%> (ø)
...de-intel-worker/internal/correlation/lsif/hover.go 81.81% <81.81%> (ø)
...se-code-intel-worker/internal/correlation/group.go 88.17% <88.17%> (ø)
... and 14 more Created by: efritz
This requires that the following PRs are merged first: