Uploaded image for project: 'Infinispan'
  1. Infinispan
  2. ISPN-1369

Make c3p0 an optional dependency of cachstore-jdbc

    XMLWordPrintable

Details

    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.

      Attachments

        Activity

          People

            ttarrant@redhat.com Tristan Tarrant
            pferraro@redhat.com Paul Ferraro
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: