-
Notifications
You must be signed in to change notification settings - Fork 68
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
fix: captured and added FileInfo query costs to metrics and hbar ratelimit class #2822
fix: captured and added FileInfo query costs to metrics and hbar ratelimit class #2822
Conversation
… limit class Signed-off-by: Logan Nguyen <[email protected]> Revert "fix: captured and added FileInfo query costs to metrics and hbar rate limit class" This reverts commit 3fe8bd80de5763b9d2803a0bb0e426f7c8ed2228. Reapply "fix: captured and added FileInfo query costs to metrics and hbar rate limit class" This reverts commit 8037a7a3baae152d18339e1589ddca37e12bd665. s Signed-off-by: Logan Nguyen <[email protected]>
Quality Gate passedIssues Measures |
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.
lg
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.
Good work. One nit and a code coverage warning. Can that be covered by a test?
The additional code was added to pretty much modify the code layout, but the overall flow remains unchanged. There are already acceptance and unit tests for that. I believe the warning is sort of based on only to the newly added code. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2822 +/- ##
==========================================
+ Coverage 81.12% 81.29% +0.17%
==========================================
Files 46 46
Lines 3386 3385 -1
Branches 707 706 -1
==========================================
+ Hits 2747 2752 +5
+ Misses 418 408 -10
- Partials 221 225 +4
Flags with carried forward coverage won't be shown. Click here to find out more.
|
…limit class (#2822) fix: captured and added FileInfo query costs to metrics and hbar rate limit class Revert "fix: captured and added FileInfo query costs to metrics and hbar rate limit class" This reverts commit 3fe8bd80de5763b9d2803a0bb0e426f7c8ed2228. Reapply "fix: captured and added FileInfo query costs to metrics and hbar rate limit class" This reverts commit 8037a7a3baae152d18339e1589ddca37e12bd665. s Signed-off-by: Logan Nguyen <[email protected]>
Description:
captured and added FileInfo query costs to metrics and hbar ratelimit class
Related issue(s):
Fixes #2738
Notes for reviewer:
Checklist