Skip to content
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

fix: high RPC usage in dispatcher #284

Merged
merged 1 commit into from
Jan 29, 2024
Merged

fix: high RPC usage in dispatcher #284

merged 1 commit into from
Jan 29, 2024

Conversation

gligneul
Copy link
Contributor

Close #278

I tested on Sepolia with Alchemy and it solves the issue.
Screenshot from 2024-01-24 18-15-19

@gligneul gligneul added bug Something isn't working ! High priority labels Jan 24, 2024
@gligneul gligneul added this to the 1.3.0 milestone Jan 24, 2024
@gligneul gligneul self-assigned this Jan 24, 2024
@gligneul gligneul added the #feat:dispatcher Feature: dispatcher label Jan 24, 2024
endersonmaia
endersonmaia previously approved these changes Jan 25, 2024
GCdePaula
GCdePaula previously approved these changes Jan 29, 2024
Copy link
Contributor

@GCdePaula GCdePaula left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Left a few unnecessary comments.

offchain/types/src/foldables.rs Outdated Show resolved Hide resolved
offchain/types/src/foldables.rs Show resolved Hide resolved
Base automatically changed from fix/input-reading to main January 29, 2024 14:07
@gligneul gligneul dismissed stale reviews from GCdePaula and endersonmaia January 29, 2024 14:07

The base branch was changed.

Copy link
Contributor

@GCdePaula GCdePaula left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀🚀🚀

@gligneul gligneul merged commit 7f0b33f into main Jan 29, 2024
6 checks passed
@gligneul gligneul deleted the fix/input-reading-2 branch January 29, 2024 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
! High priority bug Something isn't working #feat:dispatcher Feature: dispatcher
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Node make thousands of RPC requests when syncing
3 participants