Skip to content

Conversation

@eyalkoren
Copy link
Contributor

@eyalkoren eyalkoren commented Jul 9, 2019

closes #710

@eyalkoren eyalkoren self-assigned this Jul 9, 2019
@eyalkoren eyalkoren requested a review from felixbarny July 9, 2019 15:24
Copy link
Member

@felixbarny felixbarny left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing changelog (and supported-technologies update?), otherwise LGTM

@eyalkoren eyalkoren force-pushed the jms-spring-support branch from c9b2a6f to 00964bc Compare July 17, 2019 09:24
@eyalkoren
Copy link
Contributor Author

@felixbarny Added to CHANGELOG, but I am not sure about adding to supported technologies.
I assume most users utilizing this Spring feature assume this is standard JMS..

@codecov-io
Copy link

Codecov Report

Merging #714 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #714   +/-   ##
=========================================
  Coverage     61.75%   61.75%           
  Complexity       94       94           
=========================================
  Files           202      202           
  Lines          8083     8083           
  Branches       1005     1005           
=========================================
  Hits           4992     4992           
  Misses         2786     2786           
  Partials        305      305

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 916821c...00964bc. Read the comment docs.

@eyalkoren eyalkoren merged commit b36f20a into elastic:master Jul 17, 2019
@eyalkoren eyalkoren deleted the jms-spring-support branch July 17, 2019 10:06
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.

Should add support for instrumenting the listeners with parameter as subTypes of javax.jms.Message (i.e. javax.jms.MapMessage)

3 participants