Open irenjj opened 2 days ago
The changes across multiple files primarily focus on renaming and restructuring configuration structures related to Kafka and WAL (Write-Ahead Logging) setups. Key modifications include replacing the StandaloneWalConfig
with DatanodeWalConfig
and adding a KafkaTopicConfig
struct to centralize Kafka topic configurations. This update standardizes configurations and improves code maintainability by consolidating related settings under a single configuration struct.
File(s) | Change Summary |
---|---|
src/cmd/src/standalone.rs |
Renamed StandaloneWalConfig to DatanodeWalConfig in various declarations. |
src/cmd/tests/load_config_test.rs |
Replaced StandaloneWalConfig with DatanodeWalConfig in tests. |
src/common/wal/src/config.rs |
Removed StandaloneWalConfig , updated references to DatanodeWalConfig , and modified test cases accordingly. |
src/common/wal/src/config/kafka/datanode.rs |
Added multiple fields to DatanodeKafkaConfig and updated the Default implementation. |
src/common/meta/src/wal_options_allocator/... |
Updated TopicManager implementation to use nested KafkaTopicConfig instead of separate fields. |
src/common/wal/src/config/kafka/common.rs |
Introduced KafkaTopicConfig struct and added fields for topic configurations, including num_topics , selector_type , and more. |
src/common/wal/src/config/kafka/metasrv.rs |
Added KafkaTopicConfig to MetasrvKafkaConfig and removed redundant fields. |
config/config.md |
Renamed Kafka topic configuration keys with a kafka_topic_ prefix. |
tests-integration/src/tests/test_util.rs |
Refactored MetasrvWalConfig initialization to use KafkaTopicConfig . |
tests-integration/tests/region_migration.rs |
Refactored to introduce kafka_topic field in MetasrvKafkaConfig and remove individual topic-related fields. |
tests/conf/metasrv-test.toml.template |
Renamed configuration parameters for Kafka topics, e.g., num_topics to kafka_topic_num_topics . |
The requested section is skipped as the changeset does not significantly modify or introduce new control flow mechanisms that necessitate a sequence diagram.
Objective | Addressed | Explanation |
---|---|---|
Remove the StandaloneKafkaConfig struct (#4159) |
✅ |
In the land of code, changes sweep,
Old configs depart, new ones we keep.
Kafka topics now well-defined,
WithDatanodeWalConfig
, our code aligned.
Goodbye,Standalone
, it's time to part,
Hello, new structure, a fresh start.
📝✨
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?
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 84.68%. Comparing base (
be29e48
) to head (3481f13
). Report is 19 commits behind head on main.
I hereby agree to the terms of the GreptimeDB CLA.
Refer to a related PR or issue link (optional)
close: #4159
What's changed and what's your intention?
Remove the
StandaloneKafkaConfig
struct, and replaceStandaloneKafkaConfig
withDatanodeKafkaConfig
.Checklist
Summary by CodeRabbit
Refactor
StandaloneWalConfig
toDatanodeWalConfig
across the codebase for consistency.KafkaTopicConfig
.Configuration
num_topics
tokafka_topic_num_topics
).Documentation
config/config.md
to reflect changes in Kafka topic configuration parameters.