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

Handle errors gracefully #2

Open
manangouhari opened this issue Jun 1, 2021 · 0 comments
Open

Handle errors gracefully #2

manangouhari opened this issue Jun 1, 2021 · 0 comments
Labels
+P3 Priority 3 - Low priority - as-and-when

Comments

@manangouhari
Copy link
Member

Currently, I just let the indexer script crash if it encounters an error because the script is run every 15mins, so it doesn't matter much.

A better solution is to handle errors gracefully inside the indexer.Index function and return an error to the calling function in case an error is encountered, which could then be logged or sent via notification.
@manangouhari manangouhari added the +P3 Priority 3 - Low priority - as-and-when label Jun 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
+P3 Priority 3 - Low priority - as-and-when
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@manangouhari