-
Notifications
You must be signed in to change notification settings - Fork 26
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
[to #48] remove mount metircs #87
Conversation
Signed-off-by: zeminzhou <zeminzhou_@outlook.com>
Signed-off-by: zeminzhou <zhouzemin@ping.com>
// Sorter metrics | ||
// sorter.InitMetrics(registry) | ||
// memory.InitMetrics(registry) | ||
// unified.InitMetrics(registry) | ||
// leveldb.InitMetrics(registry) | ||
// redowriter.InitMetrics(registry) |
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.
how about just removing them instead of commenting them out?
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.
yes, we should remove them.
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.
LGTM~
|
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.
LGTM
Signed-off-by: zeminzhou zeminzhou_@outlook.com
What problem does this PR solve?
Issue Number: to #48
Problem Description:
The mount has been removed, related codes should be removed.
What is changed and how does it work?
Remove metrics code for mount.
Code changes
Check List for Tests
This PR has been tested by at least one of the following methods: