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

feature: support raft configuration center #6830

Merged
merged 71 commits into from
Oct 27, 2024
Merged

feature: support raft configuration center #6830

merged 71 commits into from
Oct 27, 2024

Conversation

LegGasai
Copy link
Contributor

@LegGasai LegGasai commented Sep 8, 2024 •
edited by xingfudeshi
Loading

  • I have registered the PR changes.

Ⅰ. Describe what this PR did

support raft configuration center

Ⅱ. Does this pull request fix one issue?

fixes #6829

Ⅲ. Why don't you add test cases (unit test/integration test)?

Ⅳ. Describe how to verify it

Ⅴ. Special notes for reviews

设计文档(Design doc)：https://c05j6bovqhh.feishu.cn/docx/JGEKd5QSDocc4bxz6PrcgLXUnIb
使用文档(Usage doc)：https://c05j6bovqhh.feishu.cn/docx/XM4Gdp3qPoDTwCxA1zbcvmY5n4f

LegendGasai added 15 commits July 14, 2024 15:51
@LegendGasai
feature: support raft config center(server)
739c947
@LegendGasai
feature: support raft config center(client)
52e53c9
@LegendGasai
bugfix: Add raft and rocksdbjni dependency in pom
fdb6cc3
@LegendGasai
optimize: Add some unit test and fix upload config bug
7aed12e
@LegendGasai
bugfix: add log and tcc prefix config items
5c5ba4a
@LegendGasai
optimize: rename db config key and add config unit tests
e048418
@LegendGasai
optimize: rocksdb multi-tenant storage optimization
8f31636
@LegendGasai
bugfix: Fix JVM exception caused by incorrect close of rocksdb's object
119e2d2
@LegendGasai
optimize: add configuration version manager in raft mode and add conf…
ec10f68 
…iguration management page in seata console UI.
@LegendGasai
optimize: add shutdown hook for ConfigStoreManager.
9cc03ed
@LegendGasai
fix: fix some bugs.
a1ce74c
@LegendGasai
Merge remote-tracking branch 'upstream/2.x' into summer-code-raft-con…
92468ec 
…fig-center
@LegendGasai
fix: add a npm dependency
9b256cb
@LegendGasai
optimize: add configuration meta in console ui
97df2c7
@LegendGasai
fix: remove some console file
841e413
@xingfudeshi
Copy link
Member

Please fix this

You have 2 PMD violations. For more details see: ~/incubator-seata/incubator-seata/config/seata-config-core/target/pmd.xml

xingfudeshi
xingfudeshi approved these changes Oct 23, 2024
View reviewed changes
Copy link
Member

@xingfudeshi xingfudeshi left a comment

Choose a reason for hiding this comment

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

LGTM

@xingfudeshi xingfudeshi merged commit b0fdff8 into apache:2.x Oct 27, 2024
7 checks passed
xingfudeshi added a commit that referenced this pull request Oct 27, 2024
@xingfudeshi
Revert "feature: support raft configuration center (#6830)"
5a3ef57 
This reverts commit b0fdff8.
@xingfudeshi xingfudeshi mentioned this pull request Oct 27, 2024
Merged
xingfudeshi added a commit that referenced this pull request Oct 27, 2024
@xingfudeshi
Revert "feature: support raft configuration center (#6830)" (#6953)
3de2382
@xingfudeshi
Copy link
Member

This PR will be merged in summer-code-raft-config-center branch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@xingfudeshi xingfudeshi xingfudeshi approved these changes

@funky-eyes funky-eyes Awaiting requested review from funky-eyes

@slievrly slievrly Awaiting requested review from slievrly

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

feature: support raft configuration center
5 participants
@LegGasai @xingfudeshi @LegendGasai @funky-eyes @lovepoem