Uploaded image for project: 'OpenJPA'
  1. OpenJPA
  2. OPENJPA-5

OpenJPA doesn't compile with JDBC 4



    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 0.9.0, 0.9.6
    • 2.0.0-M3
    • build / infrastructure
    • None


      Patrick opines:
      OpenJPA implements Statement, ResultSet, Connection, and maybe a
      couple other JDBC interfaces. See
      org.apache.openjpa.lib.jdbc.Delegating*. We do this for a number of
      reasons: to resolve database-specific bugs in a transparent fashion, to
      provide logging, to handle reference counting, etc.

      The pressing issue is that we must provide implementations of all of the
      methods in the various java.sql interfaces. The fact that we do not
      implement the new JDBC4 methods is why OpenJPA won't currently compile
      against JDK6. This is pretty easy to fix; take a look at
      org.apache.openjpa.lib.jdbc.DelegatingStatement to see how we handled
      this for JDBC3. Since we know that we never invoke the new methods, we
      can happily throw unsupported operation exceptions for the new methods.

      However, these unsupported methods do provide a challenge. While Kodo
      doesn't use any of these methods, our mechanism for implementing them is
      limiting, in that users who obtain Connections from Kodo will not be
      able to use the new JDBC3/JDBC4 methods in their own code. Ideally, we
      should provide some means for people to designate to OpenJPA that it
      should use a dynamic proxy to implement the unimplemented methods. This
      shouldn't be the default behavior, as the dynamic proxy will add
      overhead, but certainly could be desirable for some. I'll file an issue.


        1. OPENJPA-5.patch
          70 kB
          Marc Prud'hommeaux
        2. openjpa-5.patch.2.txt
          81 kB
          Pinaki Poddar
        3. openjpa-5.patch.3.txt
          99 kB
          Pinaki Poddar

        Issue Links



              Unassigned Unassigned
              clr Craig L Russell
              0 Vote for this issue
              0 Start watching this issue