Skip to content

StateMainService: Avoid sync fs-calls #16387

Closed

Description

The StorageService reads/write from/to disk in a synchronous matter. This happens at startup, meaning creating of other services and the window is blocked by it.

Activity

  1. jrieken commented on Dec 2, 2016

    @jrieken
    ContributorAuthor

    related to #15455

  2. joaomoreno commented on Dec 5, 2016

    @joaomoreno
    Contributor

    This is due to the nature of that service to not return promises in the get call.

  3. jrieken commented on Dec 5, 2016

    @jrieken
    ContributorAuthor

    João Moreno (@joaomoreno) Apart from the messed up nature of that service I see a way to improve this: (1) create the service with the data already loaded (before calling its ctor) and (2) perform writes async

  4. changed the title [-]Avoid sync fs-calls at startup[/-] [+]StorageService: Avoid sync fs-calls[/+] on Dec 8, 2016
  5. removed this from the Backlog milestone on Nov 15, 2017
  6. bpasero commented on Dec 21, 2018

    @bpasero
    Contributor

    The service no longer loads sync via 7024e62. The service is still writing sync though, but I am even thinking about maybe dropping the service in favour of our new storage service.

  7. changed the title [-]StorageService: Avoid sync fs-calls[/-] [+]StateMainService: Avoid sync fs-calls[/+] on May 5, 2021
  8. added
    unreleasedPatch has not yet been released in VS Code Insiders
    insiders-releasedPatch has been released in VS Code Insiders
    and removed
    unreleasedPatch has not yet been released in VS Code Insiders
    on May 5, 2021
  9. locked and limited conversation to collaborators on Jun 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Type

No type

Projects

No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions