Skip to content
This repository was archived by the owner on Jan 31, 2024. It is now read-only.

[elastic] [WIP] Just call packages.getSizes() once. #84

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

movie-travel-code
Copy link

getSizes() is used to get the information about the architecture and
the compiler. The return value is constant for specified machine, like
amd64gc for my laptop.

@movie-travel-code movie-travel-code added enhancement New feature or request Team:Code labels Aug 30, 2019
@AppVeyorBot
Copy link

`getSizes()` is used to get the information about the architecture and
the compiler. The return value is constant for specified machine, like
`amd64gc` for my laptop.
@AppVeyorBot
Copy link

@movie-travel-code
Copy link
Author

This change is not a big win because go list -f "{{context.GOARCH}}{{context.Compiler}}" -- unsafe is not a time-consuming command.

repo #79 (comment) index time - before index time - after
kubernetes 1005.466s 1121.128s 1037.315s
tidb 58.333s 64.584s 53.96s
etcd 67.286s 75.215s 69.373s

I would like to emphasize why the language server takes longer time to index the repos than #79 (comment), since CL https://go-review.googlesource.com/c/tools/+/190179 put go env GOMOD into golistDriver, every time we call golistDriver() we will call go env GOMOD.

Although the performance improvement is not so much, I think it deserves an optimization about this issue.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request Team:Code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants