tree c0153af931b3cb8ecd0748c4e15aa7fa994e4f6c
parent f1e51e6b9437f17500263287e3b9c46707f3c8c7
author Danish Dua <danishdua@google.com> 1601329680 -0400
committer Danish Dua <danishdua@google.com> 1601411152 +0000

internal/lsp/source: run deep completions before unimported completions

Unimported completions are expensive and can use up a large portion of
completion budget just to find initial deep search candidates. This
change moves these expensive operations which search through the module
cache to after normal deep completions so we search through more useful
candidates first.

Fixes golang/go#41434
Fixes golang/go#41665

Change-Id: I6f3963f8c65c1a97833a35738d2e96420de2f6ee
Reviewed-on: https://go-review.googlesource.com/c/tools/+/257974
Run-TryBot: Danish Dua <danishdua@google.com>
gopls-CI: kokoro <noreply+kokoro@google.com>
TryBot-Result: Go Bot <gobot@golang.org>
Reviewed-by: Heschi Kreinick <heschi@google.com>
Trust: Danish Dua <danishdua@google.com>