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
Add cloud storage metrics #1848
Conversation
Signed-off-by: Steven Sheehy <steven.sheehy@hedera.com>
Codecov Report
@@ Coverage Diff @@
## master #1848 +/- ##
============================================
- Coverage 87.37% 87.07% -0.31%
- Complexity 1743 1748 +5
============================================
Files 315 315
Lines 7691 7736 +45
Branches 735 738 +3
============================================
+ Hits 6720 6736 +16
- Misses 744 773 +29
Partials 227 227
Continue to review full report at Codecov.
|
Signed-off-by: Steven Sheehy <steven.sheehy@hedera.com>
SonarCloud Quality Gate failed. |
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.
Minor improvement suggestion
hedera-mirror-importer/src/main/java/com/hedera/mirror/importer/downloader/PendingDownload.java
Show resolved
Hide resolved
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
- Add `hedera.mirror.importer.cloud.latency` metric to compare consensus time against cloud storage creation time - Add `hedera.mirror.download.latency` metric to all stream types - Add `hedera.mirror.stream.close.latency` metric to all stream types - Fix event file not persisting due to missing required consensus end field - Fix cloud storage response size and time metrics not counting `_sig.gz` files as signature files Signed-off-by: Steven Sheehy <steven.sheehy@hedera.com> Signed-off-by: Ian Jungmann <ian.jungmann@hedera.com>
Detailed description:
hedera.mirror.importer.cloud.latency
metric to compare consensus time against cloud storage creation timehedera.mirror.download.latency
metric to all stream typeshedera.mirror.stream.close.latency
metric to all stream types_sig.gz
files as signature filesWhich issue(s) this PR fixes:
Special notes for your reviewer:
Checklist