Skip to content

fix: implement consistent error handling in geyser runloop#489

Merged
MicaiahReid merged 2 commits intosolana-foundation:mainfrom
dvansari65:fix/consistent-error-handling
Jan 20, 2026
Merged

fix: implement consistent error handling in geyser runloop#489
MicaiahReid merged 2 commits intosolana-foundation:mainfrom
dvansari65:fix/consistent-error-handling

Conversation

@dvansari65
Copy link
Copy Markdown
Contributor

@dvansari65 dvansari65 commented Jan 17, 2026

  • Added centralized error logging helpers (log_error, log_warn, log_info)
  • Fixed IPC server accept to properly handle failures
  • Changed silent channel send failures to propagate errors
  • Added caller notification in LoadConfig, UnloadPlugin, ReloadPlugin
  • Added context to error messages (UUID, slot numbers)
  • Fixed reload operation to be atomic (both steps succeed or fail)
  • Added logging when indexing is disabled
  • Improved transaction indexing error messages

This ensures all errors are handled consistently and critical failures
don't go unnoticed."

Copy link
Copy Markdown
Collaborator

@MicaiahReid MicaiahReid left a comment

Choose a reason for hiding this comment

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

Nice, thanks @dvansari65! Can you run cargo +nightly fmt --all to format?

@MicaiahReid MicaiahReid merged commit d24e2c7 into solana-foundation:main Jan 20, 2026
4 checks passed
@dvansari65 dvansari65 deleted the fix/consistent-error-handling branch March 20, 2026 18:07
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.

2 participants