[infinispan-issues] [JBoss JIRA] Updated: (ISPN-1369) Make c3p0 an optional dependency of cachstore-jdbc

[Paul Ferraro (JIRA)]

     [ https://issues.jboss.org/browse/ISPN-1369?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Paul Ferraro updated ISPN-1369:
-------------------------------

    Description: 
Since c3p0 is only used for pooled connection factories, it should not be a required runtime dependency.  Specifically, AS7 would rather not pull in c3p0 since AS7 will always use this cache store in conjunction with a managed connection factory.  To make c3p0 optional, c3p0 classes should only be imported by the PooledConnectionFactory itself.  Currently, there are c3p0 import statements within org.infinispan.loaders.jdbc.logging.Log which prevent c3p0 from being "optional".

N.B. To indicate an optional dependency in maven:
<dependency>
  <groupId>c3p0</groupId>
  <artifactId>c3p0</artifactId>
  <version>...</version>
  <optional>true</optional>
</dependency>

This will prevent c3p0 from being treated as a transient dependency of org.infinispan:infinispan-cachestore-jdbc.

  was:
Since c3p0 is only used for pooled connection factories, it should not be a required runtime dependency.  Specifically, AS7 would rather not pull in c3p0 since AS7 will always use this cache store in conjunction with a managed connection factory.  To make c3p0 optional, c3p0 classes should only be imported by the PooledConnectionFactory itself.  Currently, there are c3p0 import statements within org.infinispan.loaders.jdbc.logging.Log which prevent c3p0 from being "optional".

N.B. To indicate a optional dependency in maven:
<dependency>
  <groupId>c3p0</groupId>
  <artifactId>c3p0</artifactId>
  <version>...</version>
  <optional>true</optional>
</dependency>

This will prevent c3p0 from being treated as a transient dependency of org.infinispan:infinispan-cachestore-jdbc.



> Make c3p0 an optional dependency of cachstore-jdbc
> --------------------------------------------------
>
>                 Key: ISPN-1369
>                 URL: https://issues.jboss.org/browse/ISPN-1369
>             Project: Infinispan
>          Issue Type: Task
>          Components: Loaders and Stores
>    Affects Versions: 5.0.0.FINAL
>            Reporter: Paul Ferraro
>            Assignee: Manik Surtani
>
> Since c3p0 is only used for pooled connection factories, it should not be a required runtime dependency.  Specifically, AS7 would rather not pull in c3p0 since AS7 will always use this cache store in conjunction with a managed connection factory.  To make c3p0 optional, c3p0 classes should only be imported by the PooledConnectionFactory itself.  Currently, there are c3p0 import statements within org.infinispan.loaders.jdbc.logging.Log which prevent c3p0 from being "optional".
> N.B. To indicate an optional dependency in maven:
> <dependency>
>   <groupId>c3p0</groupId>
>   <artifactId>c3p0</artifactId>
>   <version>...</version>
>   <optional>true</optional>
> </dependency>
> This will prevent c3p0 from being treated as a transient dependency of org.infinispan:infinispan-cachestore-jdbc.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira