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

feat: gracefully shutdown #1210

Merged
merged 2 commits into from
Jun 26, 2024
Merged

feat: gracefully shutdown #1210

merged 2 commits into from
Jun 26, 2024

Conversation

jhult
Copy link
Collaborator

@jhult jhult commented Jun 23, 2024

Fixes #767 and #1191

@robinbb
Copy link
Member

robinbb commented Jun 25, 2024

@jhult The CI run seems to have halted in a strange place. I'll wait a few more minutes, then I'll cancel that CI job.

@jhult jhult self-assigned this Jun 25, 2024
@jhult jhult force-pushed the feat/767 branch 2 times, most recently from 8b87c94 to 0d17b53 Compare June 26, 2024 18:13
@jhult jhult requested a review from Isaac-DeFrain June 26, 2024 18:44
@jhult jhult marked this pull request as ready for review June 26, 2024 18:49
@trevorbernard
Copy link
Collaborator

trevorbernard commented Jun 26, 2024

This doesn't handle the canceling of ingestion but it satisfies the criterion for a good patch. Merging

@trevorbernard trevorbernard merged commit 6384d8c into Granola-Team:main Jun 26, 2024
2 checks passed
@jhult jhult deleted the feat/767 branch June 27, 2024 00:14
robinbb added a commit that referenced this pull request Jun 27, 2024
@jhult jhult mentioned this pull request Jul 2, 2024
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.

Indexer does not always shutdown cleanly
3 participants