Skip to content

[ISSUE #7263& ISSUE #7264] consumption offset data and topic consumerqueue not deleted on the standby node #7273

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
Open
frinda wants to merge 5 commits into
base: develop
Choose a base branch
from
Open

[ISSUE #7263& ISSUE #7264] consumption offset data and topic consumerqueue not deleted on the standby node #7273

frinda wants to merge 5 commits into from
+6 −1

Conversation

@frinda
Copy link
Contributor

@frinda frinda commented Aug 28, 2023 •
edited
Loading

Which Issue(s) This PR Fixes

Fixes #7263 and #7264

Brief Description

How Did You Test This Change?

w30003204 and others added 4 commits January 5, 2023 09:01
[ISSUE apache#5816] fix the issue that No response is returned when n…
fcd6ac3 
…o parameter is configured for the brokerStatus command.(apache#5818)
@frinda
Merge branch 'apache:develop' into develop
d988f29
[ISSUE apache#7263] fix delete synchronously the consumption offset d…
54fdbae 
…ata of the subscription group on the standby node, when execute command 'mqadmin deleteSubGroup -b xxx -g xxx -r true'.
[ISSUE apache#7264] Fix to delete synchronously topic consume queue o…
5aea257 
…n the standby node, when execute command 'mqadmin deleteTopic -b xxx -t xxx to delete unused topic'.
@frinda frinda changed the title [ISSUE #7263] fix delete synchronously the consumption offset data of the subscription group on the standby node, when execute command 'mqadmin deleteSubGroup -b xxx -g xxx -r true'. [ISSUE #7263] Fix to delete synchronously the consumption offset data of the subscription group on the standby node, when execute command 'mqadmin deleteSubGroup -b xxx -g xxx -r true' and [ISSUE #7264]Fix to delete synchronously topic consume queue on the standby node, when execute command 'mqadmin deleteTopic -b xxx -t xxx to delete unused topic'. Aug 28, 2023
@frinda frinda changed the title [ISSUE #7263] Fix to delete synchronously the consumption offset data of the subscription group on the standby node, when execute command 'mqadmin deleteSubGroup -b xxx -g xxx -r true' and [ISSUE #7264]Fix to delete synchronously topic consume queue on the standby node, when execute command 'mqadmin deleteTopic -b xxx -t xxx to delete unused topic'. [ISSUE #7263] Fix to delete synchronously the consumption offset data of the subscription group on the standby node, when execute command 'mqadmin deleteSubGroup -b xxx -g xxx -r true'. [ISSUE #7264]Fix to delete synchronously topic consume queue on the standby node, when execute command 'mqadmin deleteTopic -b xxx -t xxx to delete unused topic'. Aug 28, 2023
@codecov-commenter
Copy link

codecov-commenter commented Aug 28, 2023 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 48.75%. Comparing base (1117ab8) to head (d9a18c1).

Additional details and impacted files 
@@              Coverage Diff              @@
##             develop    #7273      +/-   ##
=============================================
- Coverage      48.85%   48.75%   -0.10%     
+ Complexity     12769    12745      -24     
=============================================
  Files           1324     1324              
  Lines          96405    96409       +4     
  Branches       12434    12434              
=============================================
- Hits           47094    47008      -86     
- Misses         43568    43642      +74     
- Partials        5743     5759      +16

☔ 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.

joeCarf
joeCarf reviewed Aug 29, 2023
View reviewed changes
Copy link
Contributor

@joeCarf joeCarf left a comment

Choose a reason for hiding this comment

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

I think the title might be too long, suggest renaming [ISSUE xx& ISSUE xxx] Fix: xxx

@frinda frinda changed the title [ISSUE #7263] Fix to delete synchronously the consumption offset data of the subscription group on the standby node, when execute command 'mqadmin deleteSubGroup -b xxx -g xxx -r true'. [ISSUE #7264]Fix to delete synchronously topic consume queue on the standby node, when execute command 'mqadmin deleteTopic -b xxx -t xxx to delete unused topic'. [ISSUE #7263& ISSUE #7264]Fix to delete synchronously the consumption offset data of the subscription group on the standby node, when execute command 'mqadmin deleteSubGroup -b xxx -g xxx -r true' and delete synchronously topic consume queue on the standby node, when execute command 'mqadmin deleteTopic -b xxx -t xxx to delete unused topic'. Aug 30, 2023
@iamgd67
Copy link
Contributor

iamgd67 commented Nov 8, 2023

lgtm

this bothers me too. I copy master's consumerqueue data to slave and restart slave to fix

@frinda
Copy link
Contributor Author

frinda commented Nov 30, 2023

seek for integration

@frinda
Copy link
Contributor Author

frinda commented Dec 4, 2023 •
edited
Loading

I think the title might be too long, suggest renaming [ISSUE xx& ISSUE xxx] Fix: xxx

@ joeCarf It's been changed. Please submit it，Thanks

@iamgd67
Copy link
Contributor

iamgd67 commented Dec 4, 2023

I think the title might be too long, suggest renaming [ISSUE xx& ISSUE xxx] Fix: xxx

@ joeCarf It's been changed. Please submit it，Thanks

still too long, maybe this?

[ISSUE #7263& ISSUE #7264] consumption offset data and topic consumerqueue not deleted on the standby node

@frinda frinda changed the title [ISSUE #7263& ISSUE #7264]Fix to delete synchronously the consumption offset data of the subscription group on the standby node, when execute command 'mqadmin deleteSubGroup -b xxx -g xxx -r true' and delete synchronously topic consume queue on the standby node, when execute command 'mqadmin deleteTopic -b xxx -t xxx to delete unused topic'. [ISSUE #7263& ISSUE #7264] Fix to cleanup consumption offset data or topic consumerqueue on the standby node when execute command 'deleteSubGroup' or 'deleteTopic' Dec 5, 2023
@frinda
Copy link
Contributor Author

frinda commented Dec 7, 2023

I think the title might be too long, suggest renaming [ISSUE xx& ISSUE xxx] Fix: xxx

@ joeCarf It's been changed. Please submit it，Thanks

still too long, maybe this?

[ISSUE #7263& ISSUE #7264] consumption offset data and topic consumerqueue not deleted on the standby node

has changed

@frinda
Copy link
Contributor Author

frinda commented Dec 7, 2023

I think the title might be too long, suggest renaming [ISSUE xx& ISSUE xxx] Fix: xxx

@joeCarf seek for integration

@frinda frinda requested a review from joeCarf December 7, 2023 08:38
joeCarf
joeCarf approved these changes Dec 8, 2023
View reviewed changes
Copy link
Contributor

@joeCarf joeCarf left a comment

Choose a reason for hiding this comment

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

LGTM

@frinda
Copy link
Contributor Author

frinda commented Dec 9, 2023

Begging to join

@frinda
Copy link
Contributor Author

frinda commented Dec 11, 2023

@mxsm @Oliverwqcwrw Help to integrate the code.

@zhiliatom
Copy link
Contributor

zhiliatom commented Jun 4, 2024

lgtm

@frinda
Copy link
Contributor Author

frinda commented Jan 6, 2025

seek for integration

@frinda frinda changed the title [ISSUE #7263& ISSUE #7264] Fix to cleanup consumption offset data or topic consumerqueue on the standby node when execute command 'deleteSubGroup' or 'deleteTopic' [ISSUE #7263& ISSUE #7264] consumption offset data and topic consumerqueue not deleted on the standby node Jan 7, 2025
@frinda
Copy link
Contributor Author

frinda commented Jan 7, 2025

@ joeCarf @ iamgd67 review again

@iamgd67
Copy link
Contributor

iamgd67 commented Jan 7, 2025

lgtm, but I am not a commiter, don't have perge permission.

@frinda
Copy link
Contributor Author

frinda commented Jan 8, 2026

seek for integration

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

Reviewers

2 more reviewers

@iamgd67 iamgd67 iamgd67 approved these changes

@joeCarf joeCarf joeCarf approved these changes

Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Bug] The consumption offset data of the subscription group cannot be deleted synchronously on the standby node.

5 participants

@frinda @codecov-commenter @iamgd67 @zhiliatom @joeCarf