commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Marc Kannegießer (JIRA) <j...@apache.org>
Subject [jira] Updated: (DBCP-289) Allow client to set a XADatasource instance instead of only its classname in BasicManagedDataSource
Date Mon, 20 Apr 2009 10:00:48 GMT

     [ https://issues.apache.org/jira/browse/DBCP-289?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Marc Kannegießer updated DBCP-289:
----------------------------------

    Attachment: set_xadatasource_instance.patch

The given patch realizes the described suggestions

> Allow client to set a XADatasource instance instead of only its classname in BasicManagedDataSource
> ---------------------------------------------------------------------------------------------------
>
>                 Key: DBCP-289
>                 URL: https://issues.apache.org/jira/browse/DBCP-289
>             Project: Commons Dbcp
>          Issue Type: Improvement
>    Affects Versions: 1.3
>            Reporter: Marc Kannegießer
>            Priority: Minor
>         Attachments: set_xadatasource_instance.patch
>
>
> {{BasicManagedDataSource}} currently allows to set the classname of an {{XADatasource}}
to create XA-enabled connections. 
> If done so the datasource is created using the Reflection-API. 
> I think in most cases this is not enough, since DataSources usually need some kind of
configuration. In my case this would be the connection-url to the database (I use {{MysqlXADatasource}}
which wants its {{setUrl(url)}} method called). 
> For this reason i think it would be useful to let the client set the {{XADataSource}}
instance instead of its classname (optional). The same is done now with the TransactionManager-Implementation.

> For this reason i'd suggest a getter and setter as well as a new member xaDataSourceInstance
as an alternative option to the classname.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message