-
Notifications
You must be signed in to change notification settings - Fork 91
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
Replace Multicall with Balancer pool data queries #476
Conversation
…g as compare. No GyroE V2)
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.
I added just some small comments, but overall the PR looks pretty good!! 👏
Just out of curiosity, do we have a an idea on how much more efficient this solution is compared to the previous multicall solution?
...r-js/src/modules/pools/pool-types/concerns/composableStable/join.concern.integration.spec.ts
Outdated
Show resolved
Hide resolved
balancer-js/src/modules/pools/pool-types/concerns/metaStable/exit.concern.integration.spec.ts
Outdated
Show resolved
Hide resolved
I'm not actually sure how to test the efficiency as it seems to depend on internet connection and RPC provider. I was thinking the best test would be to get @lgahdl to run locally at his end and see if he still sees the RPC failures he was seeing previously (doesn't really seem an issue for me). |
I was using VM inside Windows before, but now I formatted my PC and I'm using Ubuntu directly, I'm not sure if the RPC will still fail, but I can test it |
…-queries # Conflicts: # balancer-js/src/modules/sor/pool-data/onChainData.ts
In support issue here, Luiz discovered that Alchemy/Infura sometimes timeout when doing multicall because there is so much data being fetched. We should be able to solve this by using the recently deployed more efficient custom query method.
See Contract here and deployments here