Skip to content

Add INFO total_active_defrag_time and current_active_defrag_time#9377

Merged
oranagra merged 3 commits intoredis:unstablefrom
huangzhw:defrag-info
Sep 9, 2021
Merged

Add INFO total_active_defrag_time and current_active_defrag_time#9377
oranagra merged 3 commits intoredis:unstablefrom
huangzhw:defrag-info

Conversation

@huangzhw
Copy link
Contributor

@huangzhw huangzhw commented Aug 16, 2021

Add two INFO metrics:

total_active_defrag_time:12345
current_active_defrag_time:456

current_active_defrag_time if greater than 0, means how much time has
passed since active defrag started running. If active defrag stops, this metric is reset to 0.
total_active_defrag_time means total time the fragmentation
was over the defrag threshold since the server started.

This is a followup PR for #9031

…exceeded_time

Add two INFO metrics:
total_active_defrag_exceeded_time:12345
current_active_defrag_exceeded_time:456

current_active_defrag_exceeded_time if greater than 0, means how much time
active defrag starts to run. And we are still running cycles. If active defrag
stops, this metric is reset to 0.
total_active_defrag_exceeded_time means total time allocator fragmentation
is over the threshold since server startup.
@huangzhw huangzhw changed the title Add INFO total_active_defrag_exceeded_time and current_active_defrag_… Add INFO total_defrag_exceeded_time and current_defrag_exceeded_time Aug 16, 2021
@oranagra oranagra added release-notes indication that this issue needs to be mentioned in the release notes state:major-decision Requires core team consensus labels Aug 16, 2021
@oranagra oranagra changed the title Add INFO total_defrag_exceeded_time and current_defrag_exceeded_time Add INFO total_active_defrag_time and current_active_defrag_time Aug 31, 2021
@oranagra
Copy link
Member

@redis/core-team please approve

@oranagra oranagra added the approval-needed Waiting for core team approval to be merged label Aug 31, 2021
@oranagra oranagra merged commit 216f168 into redis:unstable Sep 9, 2021
@oranagra
Copy link
Member

oranagra commented Sep 9, 2021

@huangzhw can you please make a redis-doc PR?

@huangzhw huangzhw deleted the defrag-info branch September 9, 2021 10:41
@huangzhw
Copy link
Contributor Author

huangzhw commented Sep 9, 2021

OK

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

Labels

approval-needed Waiting for core team approval to be merged release-notes indication that this issue needs to be mentioned in the release notes state:major-decision Requires core team consensus

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants