Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Startup Performance decrease when sync is turned on #8354

Closed
alexwykoff opened this issue Apr 17, 2017 · 5 comments
Closed

Startup Performance decrease when sync is turned on #8354

alexwykoff opened this issue Apr 17, 2017 · 5 comments

Comments

@alexwykoff
Copy link
Contributor

alexwykoff commented Apr 17, 2017

  • Did you search for similar issues before submitting this one?
    Yes

  • Describe the issue you encountered:
    After enabling sync, a multi-second slowdown was noticed for startup.

  • Platform (Win7, 8, 10? macOS? Linux distro?):
    OS X

  • Brave Version (revision SHA):
    0.14.1 vs 0.14.2 ( 632a9bc )

  • Steps to reproduce:

    1. With a clean profile, launch 0.14.1 with sync turned off, check the time.
    2. Turn on sync, quit and launch 0.14.1, check the time, measure the delta vs previous run. (take average of 3 runs minimum)
    3. With a clean profile, launch 0.14.2 with sync turned off, check the time.
    4. Turn on sync, quit launch 0.14.2, check the time, measure the delta vs previous run. (take average of 3 runs minimum)
  • Actual result:
    0.14.2 exhibited a 2-3 second slowdown on startup

  • Expected result:
    Startup speed should remain near constant.

  • Will the steps above reproduce in a fresh profile? If not what other info can be added?
    Yes

  • Is this an issue in the currently released version?
    Yes, however 0.14.2 rc2 has shown worse performance.

  • Can this issue be consistently reproduced?
    Yes

  • Extra QA steps:
    1.
    2.
    3.

  • Screenshot if needed:
    https://www.youtube.com/watch?v=vScmYPa8FU4

  • Any related issues:

@alexwykoff alexwykoff added this to the 0.14.2 milestone Apr 17, 2017
@alexwykoff alexwykoff self-assigned this Apr 17, 2017
@luixxiul luixxiul added the perf label Apr 17, 2017
@bbondy bbondy added the needs-info Another team member needs information from the PR/issue opener. label Apr 17, 2017
@bbondy
Copy link
Member

bbondy commented Apr 17, 2017

did this regress since 0.14.1?

@alexwykoff
Copy link
Contributor Author

ticket update in progress

@alexwykoff alexwykoff added release/blocking and removed needs-info Another team member needs information from the PR/issue opener. labels Apr 17, 2017
@srirambv
Copy link
Collaborator

Run on new Windows installation

0.14.1 Sync Off 0.14.1 Sync On 0.14.2 Sync Off 0.14.2 Sync On
05.55s 5.25s 6.85s 6.25s
05.15s 5.11s 6.44s 7.77s
05.02s 5.01s 6.47s 6.35s

@ayumi
Copy link
Contributor

ayumi commented Apr 20, 2017

Merged #8408 which addresses a sync + browser-laptop startup slowdown which is most noticeable with many bookmarks.

@bsclifton
Copy link
Member

The remaining issue (#8367) will be fixed when the next build is done (which installs the new version of the client)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

6 participants