[
https://issues.apache.org/jira/browse/DBCP-258?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12561122#action_12561122
]
Peter Michaux commented on DBCP-258:
------------------------------------
Perhaps there are threading issues I'm not considering here and the current
indirect technique in the example is a better practice?
> [PATCH] ManualPoolingDriverExample - more direct construction of driver and
> minor comment fix
> ---------------------------------------------------------------------------------------------
>
> Key: DBCP-258
> URL: https://issues.apache.org/jira/browse/DBCP-258
> Project: Commons Dbcp
> Issue Type: Improvement
> Affects Versions: 1.2.2
> Reporter: Peter Michaux
> Priority: Minor
> Attachments: ManualPoolingDriverExamplePatch.diff
>
> Original Estimate: 0.08h
> Remaining Estimate: 0.08h
>
> This example has a very indirect way of registering the driver and getting it
> from the DriverManager.
> Class.forName("org.apache.commons.dbcp.PoolingDriver");
> PoolingDriver driver = (PoolingDriver)
> DriverManager.getDriver("jdbc:apache:commons:dbcp:");
> It is much more direct and clearer to do the following
> PoolingDriver driver = new PoolingDriver();
> DriverManager.registerDriver(driver);
> -------
> There were some missing backslashes in the comments about running the example
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
