-
Notifications
You must be signed in to change notification settings - Fork 320
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
chore: add stats to dedup module #5190
Conversation
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.
minor comment regarding metric naming else LGTM
24de4e6
to
78aa028
Compare
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.
Minor comment. We can leevrage RecordDuration.
78aa028
to
9ce3a70
Compare
9ce3a70
to
6e22f97
Compare
043e83f
to
5ea87dd
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #5190 +/- ##
==========================================
- Coverage 73.23% 73.14% -0.09%
==========================================
Files 439 439
Lines 51234 51238 +4
==========================================
- Hits 37520 37477 -43
- Misses 11283 11311 +28
- Partials 2431 2450 +19 ☔ View full report in Codecov by Sentry. |
Description
Adding time stats to dedup module
Linear Ticket
Fixes PIPE-1626
Security