Closed alitet01 closed 6 years ago
Merging #10 into master will increase coverage by
8.65%
. The diff coverage is45.45%
.
@@ Coverage Diff @@
## master #10 +/- ##
==========================================
+ Coverage 8.27% 16.93% +8.65%
==========================================
Files 4 5 +1
Lines 290 378 +88
Branches 34 44 +10
==========================================
+ Hits 24 64 +40
- Misses 266 314 +48
Impacted Files | Coverage Δ | |
---|---|---|
.../main/scala/ch/cern/sparkmeasure/taskmetrics.scala | 0% <0%> (ø) |
:arrow_up: |
...main/scala/ch/cern/sparkmeasure/stagemetrics.scala | 0% <0%> (ø) |
:arrow_up: |
.../main/scala/ch/cern/sparkmeasure/pushgateway.scala | 85.1% <85.1%> (ø) |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update c2cf272...87f7acf. Read the comment docs.
Thanks for the PR. OK for me. However, I'd rather change the name of the sendReport method to something like sendReportPrometheus, this is to "open the namespace" for additional implementations of metrics sinks (for example InfluxDB, Kafka)
Added export taskmetrics and stagemetrics to the Prometheus monitoring system. Some usefull info placed in additional prometheus.md document.