Skip to content

Conversation

@felixbarny
Copy link
Member

@felixbarny felixbarny commented May 13, 2018

Adds an advice which automatically sets the transaction name according
to the handler

Adds an advice which automatically sets the transaction name according
to the handler

Signed-off-by: Felix Barnsteiner <felix.barnsteiner@elastic.co>
@codecov-io
Copy link

Codecov Report

Merging #82 into master will decrease coverage by 0.63%.
The diff coverage is 41.37%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master      #82      +/-   ##
============================================
- Coverage     75.46%   74.82%   -0.64%     
  Complexity      778      778              
============================================
  Files            86       87       +1     
  Lines          2869     2900      +31     
  Branches        270      275       +5     
============================================
+ Hits           2165     2170       +5     
- Misses          567      595      +28     
+ Partials        137      135       -2
Impacted Files Coverage Δ Complexity Δ
.../co/elastic/apm/bci/ElasticApmInstrumentation.java 100% <ø> (ø) 3 <0> (ø) ⬇️
...astic/apm/spring/webmvc/ApmHandlerInterceptor.java 0% <0%> (-72.23%) 0 <0> (-5)
...g/webmvc/SpringTransactionNameInstrumentation.java 58.06% <57.14%> (ø) 5 <5> (?)

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 a3ba319...6978dab. Read the comment docs.

@felixbarny felixbarny merged commit d153664 into elastic:master May 14, 2018
@felixbarny felixbarny deleted the spring-instrumentation branch May 14, 2018 07:33
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