Tx connections are always read-write.

src/main/java/com/samskivert/jdbc/ConnectionProvider.java

@@ -66,21 +66,21 @@ public interface ConnectionProvider
     void connectionFailed (String ident, boolean readOnly, Connection conn, SQLException error);
 
     /**
-     * Returns a connection that can be used in a transaction. See {@link #getConnection}.
+     * Returns a connection that can be used in a transaction. See {@link #getConnection}. Note:
+     * transaction connections are never read-only. What would be the point?
      */
-    Connection getTxConnection (String ident, boolean readOnly)
+    Connection getTxConnection (String ident) throws PersistenceException;
-        throws PersistenceException;
 
     /**
      * Releases a connection obtained by {@link #getTxConnection}. See {@link #releaseConnection}.
      */
-    void releaseTxConnection (String ident, boolean readOnly, Connection conn);
+    void releaseTxConnection (String ident, Connection conn);
 
     /**
      * Reports failure of a connection obtained by {@link #getTxConnection}. See {@link
      * #connectionFailed}.
      */
-    void txConnectionFailed (String ident, boolean readOnly, Connection conn, SQLException error);
+    void txConnectionFailed (String ident, Connection conn, SQLException error);
 
     /**
      * Returns the URL associated with this database identifier. This should be the same value that

src/main/java/com/samskivert/jdbc/DataSourceConnectionProvider.java

@@ -80,26 +80,24 @@ public class DataSourceConnectionProvider implements ConnectionProvider
     }
 
     // from ConnectionProvider
-    public Connection getTxConnection (String ident, boolean readOnly)
+    public Connection getTxConnection (String ident) throws PersistenceException
-        throws PersistenceException
     {
         // our connections are pooled, so we can just get them normally
-        return getConnection(ident, readOnly);
+        return getConnection(ident, false);
     }
 
     // from ConnectionProvider
-    public void releaseTxConnection (String ident, boolean readOnly, Connection conn)
+    public void releaseTxConnection (String ident, Connection conn)
     {
         // our connections are pooled, so we can just release them normally
-        releaseConnection(ident, readOnly, conn);
+        releaseConnection(ident, false, conn);
     }
 
     // from ConnectionProvider
-    public void txConnectionFailed (String ident, boolean readOnly, Connection conn,
+    public void txConnectionFailed (String ident, Connection conn, SQLException error)
-                                    SQLException error)
     {
         // our connections are pooled, so we can just fail them normally
-        connectionFailed(ident, readOnly, conn, error);
+        connectionFailed(ident, false, conn, error);
     }
 
     // from ConnectionProvider

src/main/java/com/samskivert/jdbc/StaticConnectionProvider.java

@@ -128,20 +128,19 @@ public class StaticConnectionProvider implements ConnectionProvider
     }
 
     // from ConnectionProvider
-    public Connection getTxConnection (String ident, boolean readOnly) throws PersistenceException
+    public Connection getTxConnection (String ident) throws PersistenceException
     {
-        return getMapping(ident, readOnly).openConnection(ident, null);
+        return getMapping(ident, false).openConnection(ident, null);
     }
 
     // from ConnectionProvider
-    public void releaseTxConnection (String ident, boolean readOnly, Connection conn)
+    public void releaseTxConnection (String ident, Connection conn)
     {
         close(conn, ident);
     }
 
     // from ConnectionProvider
-    public void txConnectionFailed (
+    public void txConnectionFailed (String ident, Connection conn, SQLException error)
-        String ident, boolean readOnly, Connection conn, SQLException error)
     {
         close(conn, ident);
     }