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.

Sign up for GitHub

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

Jump to bottom

Catch Panic's and Write To Log #263

Closed
winder opened this issue Dec 13, 2020 · 0 comments
Closed

Catch Panic's and Write To Log #263

winder opened this issue Dec 13, 2020 · 0 comments
Assignees
@AlgoStephenAkiki
Labels
good first issue Good for newcomers Team Lamprey tech debt Things that need re-work for simplification / sanitization to reduce implementation overhead

Comments

@winder
Copy link
Contributor

winder commented Dec 13, 2020 •
edited by chaihoang
Loading

Summary

If a panic occurs in the Conduit main loop make sure it gets written to the log file.
@winder winder added the FDE label Dec 13, 2020
@winder winder added the good first issue Good for newcomers label Feb 12, 2021
@yaovi-a yaovi-a added the tech debt Things that need re-work for simplification / sanitization to reduce implementation overhead label May 26, 2021
@algoanne algoanne added Team Lamprey and removed FDE labels Jan 18, 2022
@chaihoang chaihoang reopened this Aug 4, 2022
AlgoStephenAkiki added a commit that referenced this issue Sep 28, 2022
@AlgoStephenAkiki
Add panic recovers
b43df96 
Resolves #263

Adds recovery functions in main loop of conduit as well as in a
go-routine in the start function of the pipeline
@AlgoStephenAkiki AlgoStephenAkiki mentioned this issue Sep 28, 2022
Merged
AlgoStephenAkiki added a commit that referenced this issue Oct 3, 2022
@AlgoStephenAkiki
Add panic recovers (#1251)
a6b7473 
* Add panic recovers

Resolves #263

Adds recovery functions in main loop of conduit as well as in a
go-routine in the start function of the pipeline

* helper function
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@AlgoStephenAkiki AlgoStephenAkiki

Labels
good first issue Good for newcomers Team Lamprey tech debt Things that need re-work for simplification / sanitization to reduce implementation overhead
Projects
None yet
Milestone
No milestone
Development

When branches are created from issues, their pull requests are automatically linked.

263-catch-panics-and-write-to-log algorand/indexer
5 participants
@winder @chaihoang @yaovi-a @AlgoStephenAkiki @algoanne