Fixes #91913 : Exclude **/.hg/store/** from the file watcher#91941
Fixes #91913 : Exclude **/.hg/store/** from the file watcher#91941bpasero merged 1 commit intomicrosoft:masterfrom
Conversation
Watching these files can cause several hours of 200% CPU after even trivial `hg` operations (numbers based on local testing). Excluding these files seems to bring down the duration to a few seconds.
|
@Yoric I am uncertain if this might break the popular mercurial VSCode extension that relies on file events from HG folders: https://github.com/mrcrowl/vscode-hg/blob/6bee18442233a1ec34fecd6033a934a3052e20ee/src/model.ts#L123 Maybe you could follow up with @mrcrowl if it would be ok to exclude |
|
With pleasure. Is there a good way to get in touch with @mrcrowl? |
|
I am not sure how active that account it, let's see if this ping comes back. Otherwise maybe a mail could work? |
|
Sorry, yeah I haven't been actively maintaining the Hg extension for a while, because my company switched to git, and my initial motivation was to encourage a switch to vscode from Visual Studio. Have been advertising for a motivated maintainer, but none has come forward. It is heavily based on an earlier version of the baked-in git scm extension. It creates a filesystem watcher I don't want to hold up your PR, so thinking you could test this quickly on a basic Hg repo. Unfortunately, I'm not going to have an opportunity to try this myself in the next short while. |
|
@mrcrowl thanks for the message 👍 @Yoric since you seem to be a client of this extension (?) could you do this:
|
|
I'm actually not a client of the extension, but I can test it! |
|
@Yoric thanks a ton |
|
@Yoric any update? |
|
So far, I haven't noticed anything bad happening, but I'll admit that I haven't checked very carefully. For some reason, the coronavirus scare seems to decrease the rhythm at which rebase my tree. |
|
Fair enough, I suggest we try it out in insiders for March and wait for feedback. |
Watching these files can cause several hours of 200% CPU after even
trivial
hgoperations (numbers based on local testing). Excludingthese files seems to bring down the duration to a few seconds.
This PR fixes #91913