microsoft / AzureStorageExplorer

Easily manage the contents of your storage account with Azure Storage Explorer. Upload, download, and manage blobs, files, queues, tables, and Cosmos DB entities. Gain easy access to manage your virtual machine disks. Work with either Azure Resource Manager or classic storage accounts, plus manage and configure cross-origin resource sharing (CORS) rules.
Creative Commons Attribution 4.0 International
377 stars 86 forks source link

Performance improvement #5409

Open craiha opened 2 years ago

craiha commented 2 years ago

Preflight Checklist

Problem

I am using Azure Storage Explorer on a machine with only two CPU's and it is really slow. It is great that you have reduced the size of Azure Storage Explorer in 1.22.1, now the next step is to focus on optimization. You can take inspiration from what has been accomplished with Notepad++.

Desired Solution

Azure Storage Explorer should be optimized so it is not so CPU intensive. The developers and testers should try to use it sometimes with a 2 CPU virtual machine. Optimizing the code also has an impact on battery life and energy consumption.

Alternatives and Workarounds

There is no workaround if your hardware is what it is...

Additional Context

No response

MRayermannMSFT commented 2 years ago

@craiha thanks for the feedback. Can you be more specific about what parts of Storage Explorer are slow for you? That would help us target our efforts. Thanks!

craiha commented 2 years ago

Hi,

It seems like most of the actions you do are rather CPU intensive. What is especially slow is when you start Storage Explorer and wait for it to connect to the Storages.

Regards, Carl

Den mån 14 feb. 2022 kl 20:04 skrev Matthew Rayermann < @.***>:

@craiha https://github.com/craiha thanks for the feedback. Can you be more specific about what parts of Storage Explorer are slow for you? That would help us target our efforts. Thanks!

— Reply to this email directly, view it on GitHub https://github.com/microsoft/AzureStorageExplorer/issues/5409#issuecomment-1039389913, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABYKAID63PD6CCWLBEHLDJLU3E7Z5ANCNFSM5NZQ47WQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you were mentioned.Message ID: @.***>

jdthorpe commented 2 years ago

I installed the Azure Storage explorer on Mac (OSX 12) on Feb 22, used it for an hour or so and closed it. A couple days later, my fan was running at and everything was super slow, and opening the activity monitor revealed that the ASE renderers were using 100% of my CPU, even after having closed the app several days earlier...

Screen Shot 2022-02-28 at 9 33 44 AM
MRayermannMSFT commented 2 years ago

Thanks @jdthorpe , coincidentally we made a change in our most recent version (1.23) which may address what you saw. We'll keep an eye out for that though and if we see it we'll investigate. That is not the type of behavior we want the app to have.