Skip to content

feat: add timeouts to prevent indexer hang during sync operations #89

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

PabloCastellano
Copy link
Member

This commit implements timeout mechanisms throughout the synchronizer and slots_processor modules to prevent indefinite hangs. Key changes include:

  • Add configurable request_timeout parameter to Synchronizer and SlotsProcessor
  • Implement timeouts for network operations with proper error handling
  • Add longer timeouts for more complex operations like block processing
  • Create appropriate error types (OperationTimeout) with detailed error messages

These changes ensure operations fail gracefully with clear error messages after timeout periods instead of hanging indefinitely, allowing the service to recover through pod restarts when network or processing issues occur.

This commit implements timeout mechanisms throughout the synchronizer and
slots_processor modules to prevent indefinite hangs. Key changes include:

- Add configurable request_timeout parameter to Synchronizer and SlotsProcessor
- Implement timeouts for network operations with proper error handling
- Add longer timeouts for more complex operations like block processing
- Create appropriate error types (OperationTimeout) with detailed error messages

These changes ensure operations fail gracefully with clear error messages
after timeout periods instead of hanging indefinitely, allowing the service
to recover through pod restarts when network or processing issues occur.
@PabloCastellano PabloCastellano requested a review from PJColombo May 19, 2025 19:30
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.

1 participant