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.

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

[Enhancement] Add incGroupAckNums and incGroupCkNums to LmqBrokerStatsManager #8942

Closed
1 task done
RongtongJin opened this issue Nov 16, 2024 · 0 comments · Fixed by #8943
Closed
1 task done

[Enhancement] Add incGroupAckNums and incGroupCkNums to LmqBrokerStatsManager #8942

RongtongJin opened this issue Nov 16, 2024 · 0 comments · Fixed by #8943

Comments

@RongtongJin
Copy link
Contributor

RongtongJin commented Nov 16, 2024

Before Creating the Enhancement Request

  • I have confirmed that this should be classified as an enhancement rather than a bug/feature.

Summary

Add incGroupAckNums and incGroupCkNums to LmqBrokerStatsManager
Add a setBrokerStatsManager method to enhance extensibility, allowing for more fine-grained control to be implemented externally.
Alternatively, we could consider discussing the removal of the LmqBrokerStatsManager class, as it appears to have limited value in my opinion.

Motivation

Add incGroupAckNums and incGroupCkNums to LmqBrokerStatsManager
Add a setBrokerStatsManager method to enhance extensibility, allowing for more fine-grained control to be implemented externally.
Alternatively, we could consider discussing the removal of the LmqBrokerStatsManager class, as it appears to have limited value in my opinion.

Describe the Solution You'd Like

Add incGroupAckNums and incGroupCkNums to LmqBrokerStatsManager

Describe Alternatives You've Considered

No

Additional Context

No

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

Successfully merging a pull request may close this issue.

1 participant