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

Scope of logs query causing Besu to crash #944

Closed
MadelineMurray opened this issue May 19, 2020 · 0 comments · Fixed by #986
Closed

Scope of logs query causing Besu to crash #944

MadelineMurray opened this issue May 19, 2020 · 0 comments · Fixed by #986
Assignees
Labels
bug Something isn't working P2 High (ex: Degrading performance issues, unexpected behavior of core features (DevP2P, syncing, etc))

Comments

@MadelineMurray
Copy link
Contributor

eth_getLogs queries that are too large or too broad can cause Besu to crash. Options to prevent this from occurring include:

  • Stop the logs query by detecting DB close exceptions or another signal
  • Adding a time limit to the query and recommending they reduce the scope if the time limit is reached.
@MadelineMurray MadelineMurray added the bug Something isn't working label May 19, 2020
@timbeiko timbeiko self-assigned this May 19, 2020
@timbeiko timbeiko added the P2 High (ex: Degrading performance issues, unexpected behavior of core features (DevP2P, syncing, etc)) label May 20, 2020
@timbeiko timbeiko removed their assignment May 20, 2020
@timbeiko timbeiko added this to the Chupacabra Sprint 65 milestone May 20, 2020
@AbdelStark AbdelStark self-assigned this May 27, 2020
@shemnon shemnon reopened this May 28, 2020
@AbdelStark AbdelStark removed their assignment Jun 2, 2020
@timbeiko timbeiko removed this from the Chupacabra Sprint 66 milestone Jun 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working P2 High (ex: Degrading performance issues, unexpected behavior of core features (DevP2P, syncing, etc))
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants