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
Support new metrics binary format for client stats #751
Support new metrics binary format for client stats #751
Conversation
c54f023
to
c6eeb94
Compare
For backward-compatibility purposes, metrics are written into both client attributes and new metrics binary.
c6eeb94
to
84e8641
Compare
709ca13
to
e6fb1ab
Compare
@mdumandag could you take another look at the PR? All comments seem to be addressed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the changes, it looks good to me
@mdumandag thanks for the review! |
@mdumandag I've pushed the workaround fix discussed in https://github.com/hazelcast/management-center/issues/4165 to include it into this PR. Please check e1786f3 |
Looks good to me |
For backward-compatibility purposes, metrics are written into both client attributes and new metrics binary. Also includes the following: * Fix client type value in statistics
Closes #556
For backward-compatibility purposes, metrics are written into both client attributes and new metrics binary.
Binary generator core PR: hazelcast/hazelcast#18066
Related MC issues: