microsoft / vscode-azurelogicapps

Visual Studio Code extension for Azure Logic Apps (Consumption)
https://marketplace.visualstudio.com/items?itemName=ms-azuretools.vscode-logicapps
MIT License
43 stars 28 forks source link

Extension causes high cpu load #149

Open EwertonJordao opened 4 years ago

EwertonJordao commented 4 years ago

:warning: Make sure to attach this file from your home-directory: :warning:C:\Users\ejordao\ms-azuretools.vscode-logicapps-unresponsive.cpuprofile.txt

Find more details here: https://github.com/microsoft/vscode/wiki/Explain-extension-causes-high-cpu-load

joechung-msft commented 4 years ago

Can you please attach the .cpuprofile.txt file to this issue? Or post a gist and link to it here? Thanks.

EwertonJordao commented 4 years ago

ms-azuretools.vscode-logicapps-unresponsive.cpuprofile.txt

joechung-msft commented 4 years ago

Thank you for sharing this profiler data. I will take a look at this as soon as I can.

joechung-msft commented 4 years ago

I took a look at the CPU profiler data. The unresponsiveness is caused by having to load and compile a large number of scripts from the azure-arm-resource dependency. We can address this by switching from azure-arm-logic and azure-arm-resource to @azure/arm-logic and @azure/arm-resources, which solve the problem by bundling their scripts.

EwertonJordao commented 4 years ago

Ok, I need to change my settings.json for this?

joechung-msft commented 4 years ago

This is a code change which will require updating the extension to use the newer modules instead of the older ones.