tree 533baea65f7296ebcdcdbae9284bfca4b0ecf089
parent 31a0a861b0b08dcbc23b243646584aeec342e49d
author Robert Findley <rfindley@google.com> 1660603305 -0400
committer Robert Findley <rfindley@google.com> 1662479345 +0000

[gopls-release-branch.0.9] internal/lsp/cache: clear shouldLoad IDs on load

CL 417576 externalized shouldLoad tracking into a map, which was used to
trigger a reload and cleared once reload completes. Unfortunately, it
overlooked the fact that we may also reload the entire workspace (via
reinitialization). In this case, we should clear newly loaded IDs from
the shouldLoad map, so that they are not subsequently loaded again.

For golang/go#54842
Updates golang/go#54473

Change-Id: I26f49552cae502644142dc4a4e946294db37f6f7
Reviewed-on: https://go-review.googlesource.com/c/tools/+/424074
Run-TryBot: Robert Findley <rfindley@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Alan Donovan <adonovan@google.com>
gopls-CI: kokoro <noreply+kokoro@google.com>
(cherry picked from commit e55fb40e67ba08fdaebb2e477544eb560f198210)
Reviewed-on: https://go-review.googlesource.com/c/tools/+/428596
Reviewed-by: Hyang-Ah Hana Kim <hyangah@gmail.com>