Skip to content

Add metrics and statistics about aggressive locking#687

Merged
MyonKeminta merged 7 commits intotikv:masterfrom
MyonKeminta:m/aggressive-locking-metrics
Feb 7, 2023
Merged

Add metrics and statistics about aggressive locking#687
MyonKeminta merged 7 commits intotikv:masterfrom
MyonKeminta:m/aggressive-locking-metrics

Conversation

@MyonKeminta
Copy link
Contributor

@MyonKeminta MyonKeminta commented Feb 3, 2023

Signed-off-by: MyonKeminta <MyonKeminta@users.noreply.github.com>
Signed-off-by: MyonKeminta <MyonKeminta@users.noreply.github.com>
Copy link
Collaborator

@sticnarf sticnarf left a comment

Choose a reason for hiding this comment

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

Only a small optional suggestion.

PrewriteAssertionUsageCounterNotExist prometheus.Counter
PrewriteAssertionUsageCounterUnknown prometheus.Counter

AggressiveLockedKeysNew prometheus.Counter
Copy link
Contributor

Choose a reason for hiding this comment

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

Shall we add brief comments explaining them? For example "derived" is vague if not diving into code.

Signed-off-by: MyonKeminta <MyonKeminta@users.noreply.github.com>
Signed-off-by: MyonKeminta <MyonKeminta@users.noreply.github.com>
@MyonKeminta MyonKeminta merged commit 50e86f7 into tikv:master Feb 7, 2023
@MyonKeminta MyonKeminta deleted the m/aggressive-locking-metrics branch February 7, 2023 04:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants