Skip to content
This repository has been archived by the owner on Jul 15, 2023. It is now read-only.

Multi-file find and replace spawns multiple processes per file #2202

Closed
wbrefvem opened this issue Dec 21, 2018 · 3 comments
Closed

Multi-file find and replace spawns multiple processes per file #2202

wbrefvem opened this issue Dec 21, 2018 · 3 comments

Comments

@wbrefvem
Copy link

Steps to Reproduce:

  1. Search workspace for a common string.
  2. Replace it with any other string.

Results:

  1. System resource usage should spike
  2. Process table should reveal a large number of go-related processes (gofmt, golint, goreturns, etc).
  3. Processes do not appear to be children of any VS Code process, so they persist even after exiting VS Code.
@ramya-rao-a
Copy link
Contributor

ramya-rao-a commented Dec 30, 2018

Looks like the save at the end of replacing is triggering all the on save related features like formatting and linting

@ramya-rao-a
Copy link
Contributor

This can be fixed by ensuring that we only run the build/lint/vet/format actions on an active document instead of each of the saved document.

I have pushed a commit for the same, thanks for reporting!

@ramya-rao-a
Copy link
Contributor

The fix for this is now out in the latest update (0.9.0) to the Go extension

@vscodebot vscodebot bot locked and limited conversation to collaborators Feb 13, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants