[GitHub] tomcat pull request: Add QueryExecutionReportInterceptor and tests...

[ttddyy <gi...@git.apache.org>]

GitHub user ttddyy opened a pull request:

    https://github.com/apache/tomcat/pull/18

    Add QueryExecutionReportInterceptor and tests to jdbc-pool

    Hello,
    
    I wrote a JDBC interceptor, `QueryExecutionReportInterceptor`, and would like to contribute it to jdbc-pool module.
    
    ### About
    The `QueryExecutionReportInterceptor` outputs all query execution related information to JULI log or system.out at the time of query(s) execution.
    
    *Feature:*
    - Configurable log level or System.out
    - Work with batch executions
    - Logs actual query parameters
    - Turn on/off query parameters logging
    - Support JSON output
    
    *The log includes:*
    - Query execution status (success/failure)
    - Query execution time
    - Statement type
    - Batch execution
    - Num of batch
    - Num of query
    - Query
    - Query parameter values
    
    ### Motivation
    I wrote a query reporting framework before, [datasource-proxy](https://github.com/ttddyy/datasource-proxy), which uses similar mechanism that jdbc-interceptor uses - using proxy to intercept query execution and do some logic.
    
    The reporting feature is very useful especially in debugging or trouble shooting.
    I think it would be more beneficial for users if this feature comes from part of tomcat-jdbc-connection-pool library. So, I rewrote my logic to match with jdbc-interceptor and enhanced a bit of it.
    
    ### Sample Output
    
    Statement:
    ```
    success:true, type:STATEMENT, batch:false, time:2, querySize:1, batchSize:0, query:[(select * from users)], params:[]
    ```
    
    Statement (Batched):
    ```
    success:true, type:STATEMENT, batch:true, time:1, querySize:2, batchSize:2, query:[(insert into users values (200, 'FOO')),(insert into users values (201, 'BAR'))], params:[]
    ```
    
    PreparedStatement:
    ```
    success:true, type:PREPARED, batch:false, time:3, querySize:1, batchSize:0, query:[(select 1 from users where id = ? and name = ?)], params:[(1=99,2=foo)]
    ```
    
    PreparedStatement (Batched):
    ```
    success:true, type:PREPARED, batch:true, time:1, querySize:1, batchSize:2, query:[(insert into users (id, name) values (?, ?))], params:[(1=100,2=foo),(1=200,2=bar)]
    ```
    
    PreparedStatement as JSON:
    ```
    {"success":true, "type":"PREPARED", "batch":true, "time”:1, "querySize":1, "batchSize":1, "query":["insert into users (id, name) values (?, ?)"], "params":[{"1":"100","2":"bar"}]}
    ```


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/ttddyy/tomcat query-report-interceptor

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/tomcat/pull/18.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #18
    
----
commit 61350dfa67e7f3414223e0988d4e07cdb66ee2f8
Author: Tadaya Tsuyukubo <ta...@ttddyy.net>
Date:   2015-03-18T05:02:05Z

    Add QueryExecutionReportInterceptor and tests to jdbc-pool

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org