From 8bb11ff117ca820938b6d899b13b3f2832ef244c Mon Sep 17 00:00:00 2001 From: Ian Cottrell Date: Fri, 19 Jul 2019 16:16:41 -0400 Subject: [PATCH] internal/lsp: fix lockup for packages with many files We should not be sending messages from within the telemetry worker. This does it in a new go routine now. Change-Id: I55e3b6df04699b8e45bc37b99997463f45ee114e Reviewed-on: https://go-review.googlesource.com/c/tools/+/186958 Run-TryBot: Ian Cottrell Reviewed-by: Rebecca Stambler TryBot-Result: Gobot Gobot --- internal/lsp/protocol/context.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/internal/lsp/protocol/context.go b/internal/lsp/protocol/context.go index 7a30c1dd..748edaf0 100644 --- a/internal/lsp/protocol/context.go +++ b/internal/lsp/protocol/context.go @@ -35,6 +35,6 @@ func logger(ctx context.Context, at time.Time, tags tag.List) bool { if entry.Error != nil { msg.Type = Error } - client.LogMessage(xcontext.Detach(ctx), msg) + go client.LogMessage(xcontext.Detach(ctx), msg) return true }