Skip to content

fix: Fix light client sync slow when fetch much transactions.#2944

Merged
yanguoyu merged 3 commits into
nervosnetwork:developfrom
yanguoyu:fix-light-fetch-tx
Nov 22, 2023
Merged

fix: Fix light client sync slow when fetch much transactions.#2944
yanguoyu merged 3 commits into
nervosnetwork:developfrom
yanguoyu:fix-light-fetch-tx

Conversation

@yanguoyu

Copy link
Copy Markdown
Contributor

Refer to Magickbase/neuron-public-issues#322

wallet.json Here is a watched wallet to test this issue. The wallet has a transaction that receives over 3000 cells' inputs. https://pudge.explorer.nervos.org/transaction/0x1731a226064e9b8b51b7238420452191872dfaf65a91ba1d4ccaec2cf3ed3b2e

We can compare the difference between the release and the current PR package.

@Keith-CY

Keith-CY commented Nov 17, 2023

Copy link
Copy Markdown
Contributor

/package
Packaging for test is done in 6901321466. @Keith-CY

Comment thread packages/neuron-wallet/src/block-sync-renderer/sync/light-connector.ts Outdated
@yanguoyu
yanguoyu requested a review from Keith-CY November 17, 2023 09:57
@yanguoyu

yanguoyu commented Nov 17, 2023

Copy link
Copy Markdown
Contributor Author

/package
Packaging for test is done in 6902650118. @yanguoyu

@Keith-CY

Copy link
Copy Markdown
Contributor

Please have a review @homura @devchenyan @silySuper

Co-authored-by: homura <homura.dev@gmail.com>
@silySuper

silySuper commented Nov 21, 2023

Copy link
Copy Markdown

/package
Packaging for test is done in 6938233283. @silySuper

@silySuper

Copy link
Copy Markdown

release sync time: about 3 hours when sync schedule reach to 42%
PR sync time: about 2 hours 20min when sync schedule reach to 100%
verified.

@yanguoyu
yanguoyu added this pull request to the merge queue Nov 22, 2023
Merged via the queue into nervosnetwork:develop with commit 1f53a28 Nov 22, 2023
@yanguoyu
yanguoyu deleted the fix-light-fetch-tx branch November 22, 2023 08:10
@Keith-CY Keith-CY mentioned this pull request Nov 23, 2023
@Keith-CY Keith-CY mentioned this pull request Dec 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants