Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add trace support when using JDBC CallableStatements #1541

Closed
Xylus opened this issue Feb 16, 2016 · 0 comments · Fixed by #1555
Closed

Add trace support when using JDBC CallableStatements #1541

Xylus opened this issue Feb 16, 2016 · 0 comments · Fixed by #1555
Assignees
@Xylus
Labels
enhancement
Milestone
1.5.2

Comments

@Xylus
Copy link
Contributor

Xylus commented Feb 16, 2016

Current JDBC tracers do not propagate server information when using CallableStatement, and results in rendering an unknown_db node. Adding an interceptor for CallableStatement similar to what's done for PrepareStatement should fix the issue.
@Xylus Xylus self-assigned this Feb 16, 2016
@Xylus Xylus added this to the 1.5.2 milestone Feb 16, 2016
Xylus added a commit to Xylus/pinpoint that referenced this issue Feb 25, 2016
@Xylus
pinpoint-apm#1541 Add trace support for callable statements
c8d1376
Xylus added a commit to Xylus/pinpoint that referenced this issue Feb 25, 2016
@Xylus
pinpoint-apm#1541 Add callable statement tracing to jdbc plugins
7b183b0
Xylus added a commit to Xylus/pinpoint that referenced this issue Feb 25, 2016
@Xylus
pinpoint-apm#1541 Add tests for MariaDB stored procedure calls
471a798
@Xylus Xylus mentioned this issue Feb 25, 2016
Merged
dawidmalina pushed a commit to dawidmalina/pinpoint that referenced this issue Mar 20, 2016
@Xylus @dawidmalina
pinpoint-apm#1541 Add trace support for callable statements
8a9ec4e
dawidmalina pushed a commit to dawidmalina/pinpoint that referenced this issue Mar 20, 2016
@Xylus @dawidmalina
pinpoint-apm#1541 Add callable statement tracing to jdbc plugins
a07a510
dawidmalina pushed a commit to dawidmalina/pinpoint that referenced this issue Mar 20, 2016
@Xylus @dawidmalina
pinpoint-apm#1541 Add tests for MariaDB stored procedure calls
1d44810
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Xylus Xylus

Labels
enhancement
Projects
None yet
Milestone
1.5.2
Development

Successfully merging a pull request may close this issue.

Adds callable statement tracing for JDBC libraries Xylus/pinpoint
1 participant
@Xylus