Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

[ISSUE #9777 ]feat: use data version from master while sync slave and fix delete co…#9778

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
absolute8511 wants to merge6 commits intoapache:develop
base:develop
Choose a base branch
Loading
fromabsolute8511:fix-sync-slave

Conversation

@absolute8511
Copy link
Contributor

…nfig while sync

Change-Id: I42b2e7b1acc6836d3c90973801c9defba5f1325c

Which Issue(s) This PR Fixes

Fixes#9777

Brief Description

How Did You Test This Change?

…nfig while syncChange-Id: I42b2e7b1acc6836d3c90973801c9defba5f1325c
@codecov-commenter
Copy link

codecov-commenter commentedOct 23, 2025
edited
Loading

Codecov Report

❌ Patch coverage is22.72727% with17 lines in your changes missing coverage. Please review.
✅ Project coverage is 48.29%. Comparing base (2eb2129) to head (9838025).

Files with missing linesPatch %Lines
...ker/config/v1/RocksDBSubscriptionGroupManager.java0.00%6 Missing⚠️
...mq/broker/config/v1/RocksDBTopicConfigManager.java0.00%6 Missing⚠️
...ocketmq/broker/config/v1/RocksDBConfigManager.java0.00%4 Missing⚠️
...apache/rocketmq/broker/slave/SlaveSynchronize.java75.00%1 Missing⚠️
Additional details and impacted files
@@              Coverage Diff              @@##             develop    #9778      +/-   ##=============================================- Coverage      48.32%   48.29%   -0.04%+ Complexity     12234    12224      -10=============================================  Files           1313     1313                Lines          93639    93655      +16       Branches       12006    12006              =============================================- Hits           45254    45226      -28- Misses         42788    42824      +36- Partials        5597     5605       +8

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report?Share it here.

🚀 New features to boost your workflow:
  • ❄️Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@absolute8511absolute8511 changed the titlefeat: use data version from master while sync slave and fix delete co…[#ISSUE9777 ]feat: use data version from master while sync slave and fix delete co…Oct 23, 2025
@absolute8511absolute8511 changed the title[#ISSUE9777 ]feat: use data version from master while sync slave and fix delete co…[ISSUE #9777 ]feat: use data version from master while sync slave and fix delete co…Oct 23, 2025
Change-Id: I7ec20607a84499fe5a6607763013c59d726aedc3
Change-Id: Ic845794350e8bdaa847bdd0ae4b3e40ab1ad6311
Change-Id: I39e78477a5223b578a4ede3e5cb76f04368d1ca3
Change-Id: Iebe28ffcd7946ee865daf1a8f01f43483244d541
Change-Id: I9e835568912928ddf6e81816095ee3ed8f93afc0
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@RongtongJinRongtongJinRongtongJin approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Bug] Should not update topic config data version while sync slave use master instead

3 participants

@absolute8511@codecov-commenter@RongtongJin

[8]ページ先頭

©2009-2025 Movatter.jp