Skip to content

Conversation

@felixbarny
Copy link
Member

@felixbarny felixbarny commented Jul 5, 2018

It can take quite a lot of space, as it's added to each transaction, and stored even for non-sampled transactions

It can take quite a lot of space, as it's added to each transaction,
and stored even for non-sampled transactions
@codecov-io
Copy link

Codecov Report

Merging #118 into master will increase coverage by 0.08%.
The diff coverage is 87.5%.

Impacted file tree graph

@@            Coverage Diff             @@
##             master   #118      +/-   ##
==========================================
+ Coverage     76.92%    77%   +0.08%     
- Complexity      953    955       +2     
==========================================
  Files            99     99              
  Lines          3289   3297       +8     
  Branches        290    291       +1     
==========================================
+ Hits           2530   2539       +9     
+ Misses          603    602       -1     
  Partials        156    156
Impacted Files Coverage Δ Complexity Δ
...a/co/elastic/apm/report/ReporterConfiguration.java 100% <100%> (ø) 9 <1> (+1) ⬆️
.../java/co/elastic/apm/report/ApmServerReporter.java 54.44% <50%> (-0.11%) 11 <0> (ø)
...a/co/elastic/apm/report/ReportingEventHandler.java 81.81% <0%> (+6.06%) 10% <0%> (+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 d765466...f05a7dd. Read the comment docs.

@felixbarny felixbarny merged commit fd0adee into elastic:master Jul 5, 2018
@felixbarny felixbarny deleted the exclude-argv-by-default branch July 5, 2018 07:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants