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.

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

JDBC Telemetry on explain plan is filter on uppercase "SELECT" only #907

Closed
cy33hc opened this issue Apr 12, 2019 · 1 comment · Fixed by #940
Closed

JDBC Telemetry on explain plan is filter on uppercase "SELECT" only #907

cy33hc opened this issue Apr 12, 2019 · 1 comment · Fixed by #940
Labels

Comments

@cy33hc
Copy link

cy33hc commented Apr 12, 2019

Expected behavior

JDBC telemetry should filter on query with mixed case "SELECT" cmd

Actual behavior

Filtering only on uppercase "SELECT"

To Reproduce

enable jdbc telemetry in AI-Agent.xml. Then create a slow jdbc query with lowercase "select" to activate the explain plan.

Sample Application

See the code from application insights.

https://github.com/Microsoft/ApplicationInsights-Java/blob/master/core/src/main/java/com/microsoft/applicationinsights/internal/agent/CoreAgentNotificationsHandler.java

image

@dhaval24
Copy link
Contributor

dhaval24 commented Apr 16, 2019

@cy33hc thanks for opening this issue! This is a bug and should be fixed! Please feel free to file a PR if you would love to contribute to the project. We are always willing to accept contributions and really appreciate them.

@ghost ghost locked as resolved and limited conversation to collaborators Jul 20, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants