[ 
https://issues.apache.org/jira/browse/OAK-4477?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15374828#comment-15374828
 ] 

Julian Reschke edited comment on OAK-4477 at 7/14/16 12:20 PM:
---------------------------------------------------------------

Seems to be caused by https://bz.apache.org/bugzilla/show_bug.cgi?id=59850 -- 
the StatementCache appears to break the JDBC contract about when ResultSets are 
automatically closed - see OAK-4559.

Resolve *this* bug by defaulting to all interceptors expect StatementCache, and 
allowing to override this using a system property for testing.




was (Author: reschke):
Seems to be caused by https://bz.apache.org/bugzilla/show_bug.cgi?id=59850 -- 
the StatementCache appears to break the JDBC contract about when ResultSets are 
automatically closed.

> RDBDatasourceFactory should use pool config similar to sling datasource 
> defaults
> --------------------------------------------------------------------------------
>
>                 Key: OAK-4477
>                 URL: https://issues.apache.org/jira/browse/OAK-4477
>             Project: Jackrabbit Oak
>          Issue Type: Technical task
>          Components: rdbmk
>    Affects Versions: 1.0.31, 1.4.3, 1.5.3, 1.2.16
>            Reporter: Julian Reschke
>            Assignee: Julian Reschke
>              Labels: candidate_oak_1_0, candidate_oak_1_2, candidate_oak_1_4
>             Fix For: 1.6
>
>         Attachments: OAK-4477.diff
>
>
> {{RDBDataSourceFactory}} is used only for testing, and creates instances of 
> {{org.apache.tomcat.jdbc.pool.DataSource}}.
> These are currently created with default config, while 
> {{org.apache.sling.datasource}} (which is likely used in production) uses 
> it's own defaults. In particular, it configures three interceptors -- 
> {{StatementCache;SlowQueryReport(threshold=10000);ConnectionState}} -- which 
> we do not, thus they aren't getting unit test coverage.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to