cloudfoundry-attic / cf-abacus

CF usage metering and aggregation
Apache License 2.0
98 stars 86 forks source link

Introduce sampler storage module #1030

Closed ghost closed 6 years ago

ghost commented 6 years ago

Changes

cf-gitbot commented 6 years ago

We have created an issue in Pivotal Tracker to manage this:

https://www.pivotaltracker.com/story/show/160346531

The labels on this github issue will be updated when the story is started.

cfdreddbot commented 6 years ago

Hey momchil-sap!

Thanks for submitting this pull request! I'm here to inform the recipients of the pull request that you and the commit authors have already signed the CLA.

codecov[bot] commented 6 years ago

Codecov Report

Merging #1030 into master will increase coverage by 0.04%. The diff coverage is 97.14%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1030      +/-   ##
==========================================
+ Coverage   95.02%   95.07%   +0.04%     
==========================================
  Files         107      108       +1     
  Lines        3138     3208      +70     
==========================================
+ Hits         2982     3050      +68     
- Misses        156      158       +2
Impacted Files Coverage Δ
lib/sampler/storage/src/lib/span-dao.js 97.14% <97.14%> (ø)

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 31ebbb7...d664b56. Read the comment docs.