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

Critical errors #181

Draft
wants to merge 33 commits into
base: dev
Choose a base branch
from
Draft

Critical errors #181

wants to merge 33 commits into from

Conversation

idky137
Copy link
Contributor

@idky137 idky137 commented Feb 4, 2025

Adds automatic server restart for critical errors.

  • propagates errors from child processes to Indexer.
  • Updates Indexer::serve and Indexer::start to return IndexerErrors properly.
  • Updates main to restart on Critical errors.

PR details:

@idky137 idky137 added the ZGM2 Issues that need to be resolved for the completion of the Zaino dev grant milestone 2 label Feb 4, 2025
@idky137
Copy link
Contributor Author

idky137 commented Feb 4, 2025

This is now code complete, I will change it to ready to review once previous PRs are merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ZGM2 Issues that need to be resolved for the completion of the Zaino dev grant milestone 2
Projects
Status: In progress
Development

Successfully merging this pull request may close these issues.

Task 2.2.3: Add automatic server restart / shutdown for critical errors.
1 participant