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

Close local shards and delete mrecordlog queues on ingester startup #3895

Merged
merged 1 commit into from
Oct 6, 2023

Conversation

guilload
Copy link
Member

Description

On startup, ingesters list their local mrecordlog queues and call the metastore to close the corresponding shards. If a shard no longer exists, its queue is deleted. Closed shards whose records have been indexed are eventually deleted by a GC task after a grace period has elapsed. The metastore notifies the control plane of the close and delete shards event.

How was this PR tested?

  • Added unit tests

@guilload guilload force-pushed the guilload/close-shards-delete-queues-on-startup branch 3 times, most recently from 6c315e7 to 66916b2 Compare October 3, 2023 19:02
Copy link
Member Author

@guilload guilload left a comment

Choose a reason for hiding this comment

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

This is ready for review!

quickwit/quickwit-codegen/example/src/lib.rs Show resolved Hide resolved
quickwit/Cargo.toml Show resolved Hide resolved
quickwit/quickwit-common/src/tower/pool.rs Outdated Show resolved Hide resolved
quickwit/quickwit-proto/src/error.rs Show resolved Hide resolved
@guilload guilload force-pushed the guilload/close-shards-delete-queues-on-startup branch from 66916b2 to ff571c9 Compare October 3, 2023 19:03
@guilload guilload force-pushed the guilload/close-shards-delete-queues-on-startup branch from ff571c9 to 1f9ddbb Compare October 4, 2023 16:51
@guilload guilload mentioned this pull request Oct 4, 2023
@guilload guilload force-pushed the guilload/close-shards-delete-queues-on-startup branch from 1f9ddbb to 1de1432 Compare October 5, 2023 16:37
@guilload guilload force-pushed the guilload/close-shards-delete-queues-on-startup branch from 1de1432 to 64ea389 Compare October 6, 2023 13:57
@guilload guilload linked an issue Oct 6, 2023 that may be closed by this pull request
@guilload guilload force-pushed the guilload/close-shards-delete-queues-on-startup branch from 64ea389 to 0c7c884 Compare October 6, 2023 19:13
@guilload guilload force-pushed the guilload/close-shards-delete-queues-on-startup branch from 0c7c884 to aa0fc9f Compare October 6, 2023 19:20
@guilload guilload merged commit e05b708 into main Oct 6, 2023
4 checks passed
@guilload guilload deleted the guilload/close-shards-delete-queues-on-startup branch October 6, 2023 19:49
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.

Close local shards and GC mrecordlog queues on ingester startup
2 participants