search

microsoft / vscode-eslint

VSCode extension to integrate eslint into VSCode
MIT License
1.75k stars 335 forks source link

120GB Memory is used when using `worker_threads` in plugin 😭 #1503

Closed JounQin closed 2 years ago

JounQin commented 2 years ago

image

I just got crazy about this

Reproduction: https://github.com/un-ts/yocto-ttl-cache

Open any .ts or .json, .md file (make sure eslint.validate and eslint.options#extensions configured first).

The memory and threads are just growing up in Activity Monitor.app

https://github.com/un-ts/synckit is used by many ESLint plugins, and it just runs well with ESLint cli.

EdisonSu768 commented 2 years ago
企业微信截图_24f02306-3d5a-4ad8-878c-802736f4a7b1

I also got this. But after eslint server got EAGAIN error, the memory of vscode looks good.I wish the screenshot can make some help to debug.

JounQin commented 2 years ago

Thanks @EdisonSu768 for debugging and fixing this at https://github.com/mdx-js/eslint-mdx/pull/412!