Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

EditorNavigationContribution.onStart is slow #3413

Open
akosyakov opened this issue Nov 6, 2018 · 3 comments
Open

EditorNavigationContribution.onStart is slow #3413

akosyakov opened this issue Nov 6, 2018 · 3 comments
Labels
performance issues related to performance

Comments

@akosyakov
Copy link
Member

It takes over 1s each time.

@kittaakos kittaakos added the performance issues related to performance label Nov 6, 2018
@kittaakos
Copy link
Contributor

As we have exceeded the 8 seconds, I looked into it. It seems it is not the editor navigator contributions which is slow but waiting for the workspace root to be set for the workspace storage service. For instance, if I remove the await from here:

https://github.com/theia-ide/theia/blob/e147cf39c7f484374b7ec0cd0d04958c3ef00bca/packages/editor/src/browser/editor-navigation-contribution.ts#L79

Then I can see the bundle.js:95521 root WARN DebugFrontendApplicationContribution.onStart is slow, took: 8154.099999985192 ms warning with the same ~8 seconds.

@akosyakov
Copy link
Member Author

Can we restore navigation history async without blocking onStart?

@kittaakos
Copy link
Contributor

Yes, we can but it won't change much. The overall app startup time will remain the same. If not the EditorNavigationContribution blocks, then something else waiting for the WS root will.

According to my measurements (console logging), the bottleneck is here:
https://github.com/theia-ide/theia/blob/e147cf39c7f484374b7ec0cd0d04958c3ef00bca/packages/workspace/src/browser/workspace-service.ts#L142

kittaakos pushed a commit that referenced this issue May 10, 2019
Closes #3413

Signed-off-by: Akos Kitta <kittaakos@typefox.io>
kittaakos pushed a commit that referenced this issue May 10, 2019
Closes #3413

Signed-off-by: Akos Kitta <kittaakos@typefox.io>
kittaakos pushed a commit that referenced this issue May 10, 2019
Closes #3413

Signed-off-by: Akos Kitta <kittaakos@typefox.io>
kittaakos pushed a commit that referenced this issue May 10, 2019
Closes #3413

Signed-off-by: Akos Kitta <kittaakos@typefox.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
performance issues related to performance
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants