-
Notifications
You must be signed in to change notification settings - Fork 329
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(log_store): introduce the CollectionTask #4530
feat(log_store): introduce the CollectionTask #4530
Conversation
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configuration File (
|
9a86dff
to
3adb382
Compare
@coderabbitai summary |
Actions performedSummary regeneration triggered. |
WalkthroughThe recent changes enhance the configuration and functionality of the data node and log store systems. Key improvements include the introduction of new indexing capabilities, modifications to error handling, and refinements in object store management. These updates promote better organization, flexibility, and performance, ultimately leading to a more robust system architecture. Changes
Sequence Diagram(s)sequenceDiagram
participant ClientManager
participant KafkaLogStore
participant GlobalIndexCollector
participant DatanodeBuilder
ClientManager->>KafkaLogStore: try_new(config, global_index_collector)
KafkaLogStore->>GlobalIndexCollector: Initialize with optional collector
KafkaLogStore->>DatanodeBuilder: Build with indexing config
DatanodeBuilder->>GlobalIndexCollector: Manage indexing operations
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configuration File (
|
8977c62
to
8d7bcc9
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #4530 +/- ##
==========================================
- Coverage 84.88% 84.63% -0.25%
==========================================
Files 1094 1096 +2
Lines 196364 196628 +264
==========================================
- Hits 166674 166410 -264
- Misses 29690 30218 +528 |
660bff0
to
392c314
Compare
@fengjiachun @v0y4g3r PTAL |
73ac3b1
to
1f47698
Compare
@WenyXu CI failed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
* feat: introduce the `CollectionTask` * feat: add config of index collector * chore: remove unused code * feat: truncate indexes * chore: apply suggestions from CR * chore: update config examples * refactor: retrieve latest offset while dumping indexes * chore: print warn
I hereby agree to the terms of the GreptimeDB CLA.
Refer to a related PR or issue link (optional)
#4385
What's changed and what's your intention?
Introduce the
CollectionTask
, it performs two main operations:path
at everydump_index_interval
.checkpoint_interval
.Checklist
Summary by CodeRabbit
New Features
GlobalIndexCollector
for better index management in the data node.default_index_file
andGlobalIndexCollector
.Bug Fixes
Chores