-
-
Notifications
You must be signed in to change notification settings - Fork 183
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
Parallelize network requests in assets controllers #1801
Conversation
nice! |
I'm looking at the tests failing in CI. But so far they're passing locally. |
🚀 |
packages/assets-controllers/src/Standards/NftStandards/ERC721/ERC721Standard.ts
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
18d4131
Some NFT tests (the |
## Explanation Parallelizes network requests in assets controllers for better performance. I see each non-cached RPC request taking ~50ms on fast internet, and ~500ms on 3g internet. Parallelizing requests where possible causes a noticeable performance improvement. For example, getting details for an ERC-721 NFT in parallel benchmarks ~100ms quicker on fast internet, and ~600ms faster on 3g internet. ## Changelog ### `@metamask/assets-controllers` - **FIXED**: Parallelized network requests for better performance ## Checklist - [ ] I've updated the test suite for new or updated code as appropriate - [ ] I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate - [x] I've highlighted breaking changes using the "BREAKING" category above as appropriate --------- Co-authored-by: Alex Donesky <adonesky@gmail.com>
Explanation
Parallelizes network requests in assets controllers for better performance.
I see each non-cached RPC request taking ~50ms on fast internet, and ~500ms on 3g internet. Parallelizing requests where possible causes a noticeable performance improvement. For example, getting details for an ERC-721 NFT in parallel benchmarks ~100ms quicker on fast internet, and ~600ms faster on 3g internet.
Changelog
@metamask/assets-controllers
Checklist