live-share icon indicating copy to clipboard operation
live-share copied to clipboard

Extension has slow startup performance

Open vaibhavarora14 opened this issue 4 years ago • 6 comments

  • Issue Type: Bug
  • Extension Name: vsliveshare
  • Extension Version: 1.0.3206
  • OS Version: Windows_NT x64 10.0.19041
  • VSCode version: 1.51.1
{
	"messages": [],
	"activationTimes": {
		"codeLoadingTime": 197,
		"activateCallTime": 2,
		"activateResolvedTime": 10832,
		"activationReason": {
			"startup": true,
			"extensionId": {
				"value": "ms-vsliveshare.vsliveshare",
				"_lower": "ms-vsliveshare.vsliveshare"
			},
			"activationEvent": "*"
		}
	},
	"runtimeErrors": []
}

vaibhavarora14 avatar Nov 20 '20 12:11 vaibhavarora14

There's incredibly good discussion on some perf traces here in this issue. #4218 I'll still keep this as the primary issue though.

Davsterl avatar Jul 18 '21 22:07 Davsterl

@alyssajotice @fubaduba do you know if this is getting looked at? Performance is one of the most raised issues on this repo.

Example below

Extension Eager Load Code Call Activate Finish Activate Event By
ms-vsliveshare.vsliveshare true 62 0 3433 * ms-vsliveshare.vsliveshare

3.4 seconds is a long time for an extension to load, compared to the others which are only in the region of 50ms. Also why is Live Share "eager"? It shouldn't start up unless I'm actively starting it. Which most of the time i'm not using it.

There was already discussion of using onStartupFinished but that still hasn't been looked at.

jasonwilliams avatar Nov 24 '21 10:11 jasonwilliams

Linking this too: https://github.com/MicrosoftDocs/live-share/issues/4470.

I will start a conversation with @fubaduba about this.

alyssajotice avatar Nov 24 '21 13:11 alyssajotice

I can attempt a PR to use onStartupFinished : would it be considered? Anything to take care of when attempting this change?

aymericbouzy avatar May 06 '22 08:05 aymericbouzy

I can attempt a PR to use onStartupFinished : would it be considered? Anything to take care of when attempting this change?

@aymericbouzy the extension is closed source so I don’t know how you’d manage that.

jasonwilliams avatar May 06 '22 20:05 jasonwilliams

I hadn't realized haha 😂

aymericbouzy avatar May 07 '22 06:05 aymericbouzy

This issue has been automatically marked as stale because it has not had recent activity. It will be closed automatically in 2 days.

github-actions[bot] avatar Aug 15 '22 10:08 github-actions[bot]

Closing in favor of https://github.com/MicrosoftDocs/live-share/issues/4470 which should address this issue.

derekbekoe avatar Nov 11 '22 02:11 derekbekoe