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

PAYARA-554 modified and improved SQL Tracing #586

Merged

Conversation

smillidge
Copy link
Contributor

Execution time of the JDBC method is now recorded in the SQL Trace.
Also SQLTraceCache now also displays number of executions in the console.#
Now enable custom connection pool properties for application defined datasource using DataSourceDefinition
Removed synchronized block from SQLTraceCache and replaced with Concurrent collection
Created SlowSQLLogger to report SQL queries that exceed a threshold time.
Plumbed the SQL tracing and Slow SQL Logging into the admin console and incorporated the two in-built SQL tracers directly

…s also recorded.

Also SQLTraceCache now also displays number of executions in the console.#
Removed duplicated log info from SQLTraceLogger
Now enable custom connection pool properties for application defined datasource using DataSourceDefinition
PAYARA-554 removed synchronized block from SQLTraceCache and replaced with Concurrent collection

Initial implementation of the slow SQL logger

Forgot to add the new files

Plumbed the SQL tracing and Slow SQL Logging into the admin console and incorporated the two in-built SQL tracers directly
@smillidge
Copy link
Contributor Author

jenkins test please

@smillidge
Copy link
Contributor Author

jenkins test please

@payara-ci
Copy link
Contributor

One or more tests have failed

@smillidge
Copy link
Contributor Author

jenkins test please

@payara-ci
Copy link
Contributor

One or more tests have failed

@smillidge
Copy link
Contributor Author

jenkins test please

@payara-ci
Copy link
Contributor

One or more tests have failed

@MeroRai
Copy link
Member

MeroRai commented Jan 8, 2016

jenkins test please

1 similar comment
@smillidge
Copy link
Contributor Author

jenkins test please

@payara-ci
Copy link
Contributor

All tests have passed

smillidge added a commit that referenced this pull request Jan 8, 2016
…cing

PAYARA-554 modified and improved SQL Tracing
@smillidge smillidge merged commit 8420438 into payara:master Jan 8, 2016
@smillidge smillidge deleted the PAYARA-554-connection-pool-tracing branch January 30, 2016 07:17
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.

None yet

3 participants