You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "Lee Jun Gyun (JIRA)" <ji...@apache.org> on 2019/07/03 07:17:00 UTC
[jira] [Comment Edited] (DBCP-547) Add a ConnectionFactory
Interface in BasicDataSource class
[ https://issues.apache.org/jira/browse/DBCP-547?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16877519#comment-16877519 ]
Lee Jun Gyun edited comment on DBCP-547 at 7/3/19 7:16 AM:
-----------------------------------------------------------
I raise the PR https://github.com/apache/commons-dbcp/pull/33
was (Author: simple):
I raise the PR https://github.com/apache/commons-dbcp/pull/32
> Add a ConnectionFactory Interface in BasicDataSource class
> ----------------------------------------------------------
>
> Key: DBCP-547
> URL: https://issues.apache.org/jira/browse/DBCP-547
> Project: Commons DBCP
> Issue Type: Improvement
> Affects Versions: 2.2.0
> Reporter: Lee Jun Gyun
> Priority: Minor
> Time Spent: 50m
> Remaining Estimate: 0h
>
> It think it would be nice better to implement a ConnectionFactory interface in BasicDataSource class or implement a interface in DriverConnectionFactory class.
> In my current project, when getting a connection in my program, I call oracleSetModule.
> so, i think that after creating connection, call oracleSetModule via connectionFactoryImpl seems to be nice.
>
> [Before]
> protected ConnectionFactory createConnectionFactory() throws SQLException {
> ...
> ...
> ConnectionFactory driverConnectionFactory =
> new DriverConnectionFactory(driverToUse, url, connectionProperties);
> return driverConnectionFactory;
> }
> [After]
> protected ConnectionFactory createConnectionFactory() throws SQLException {
> ...
> ...
> ConnectionFactory driverConnectionFactory =
> getConnectionFactoryImpl().newInstance(...);
> return driverConnectionFactory;
> }
>
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)