Author: reschke
Date: Wed Mar 9 15:58:04 2016
New Revision: 1734279
URL: http://svn.apache.org/viewvc?rev=1734279&view=rev
Log:
OAK-3977: RDBDocumentStore: remove PostgreSQL workaround after upgrading to
JDBC driver 9.4.1208
Modified:
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/document/rdb/RDBBlobStoreDB.java
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/document/rdb/RDBDocumentStoreDB.java
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/document/rdb/RDBDocumentStoreJDBC.java
jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/document/rdb/RDBDocumentStoreJDBCTest.java
Modified:
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/document/rdb/RDBBlobStoreDB.java
URL:
http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/document/rdb/RDBBlobStoreDB.java?rev=1734279&r1=1734278&r2=1734279&view=diff
==============================================================================
---
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/document/rdb/RDBBlobStoreDB.java
(original)
+++
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/document/rdb/RDBBlobStoreDB.java
Wed Mar 9 15:58:04 2016
@@ -96,7 +96,7 @@ public enum RDBBlobStoreDB {
POSTGRES("PostgreSQL") {
@Override
public String checkVersion(DatabaseMetaData md) throws SQLException {
- return RDBJDBCTools.versionCheck(md, 9, 3, description);
+ return RDBJDBCTools.versionCheck(md, 9, 5, 9, 4, description);
}
@Override
Modified:
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/document/rdb/RDBDocumentStoreDB.java
URL:
http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/document/rdb/RDBDocumentStoreDB.java?rev=1734279&r1=1734278&r2=1734279&view=diff
==============================================================================
---
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/document/rdb/RDBDocumentStoreDB.java
(original)
+++
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/document/rdb/RDBDocumentStoreDB.java
Wed Mar 9 15:58:04 2016
@@ -77,7 +77,32 @@ public enum RDBDocumentStoreDB {
POSTGRES("PostgreSQL") {
@Override
public String checkVersion(DatabaseMetaData md) throws SQLException {
- return RDBJDBCTools.versionCheck(md, 9, 3, description);
+ String result = RDBJDBCTools.versionCheck(md, 9, 5, 9, 4,
description);
+
+ if (result.isEmpty()) {
+ // special case: we need 9.4.1208 or newer (see OAK-3977)
+ if (md.getDriverMajorVersion() == 9 &&
md.getDriverMinorVersion() == 4) {
+ String versionString = md.getDriverVersion();
+ String scanfor = "9.4.";
+ int p = versionString.indexOf(scanfor);
+ if (p >= 0) {
+ StringBuilder build = new StringBuilder();
+ for (char c : versionString.substring(p +
scanfor.length()).toCharArray()) {
+ if (c >= '0' && c <= '9') {
+ build.append(c);
+ } else {
+ break;
+ }
+ }
+ if (Integer.parseInt(build.toString()) < 1208) {
+ result = "Unsupported " + description + " driver
version: " + md.getDriverVersion() + ", found build "
+ + build + ", but expected at least build
1208";
+ }
+ }
+ }
+ }
+
+ return result;
}
@Override
Modified:
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/document/rdb/RDBDocumentStoreJDBC.java
URL:
http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/document/rdb/RDBDocumentStoreJDBC.java?rev=1734279&r1=1734278&r2=1734279&view=diff
==============================================================================
---
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/document/rdb/RDBDocumentStoreJDBC.java
(original)
+++
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/document/rdb/RDBDocumentStoreJDBC.java
Wed Mar 9 15:58:04 2016
@@ -302,11 +302,7 @@ public class RDBDocumentStoreJDBC {
results = stmt.executeBatch();
} catch (BatchUpdateException ex) {
LOG.debug("Some of the batch updates failed", ex);
- if (this.dbInfo == RDBDocumentStoreDB.POSTGRES) {
- results = new int[0];
- } else {
- results = ex.getUpdateCounts();
- }
+ results = ex.getUpdateCounts();
} finally {
stmt.close();
}
Modified:
jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/document/rdb/RDBDocumentStoreJDBCTest.java
URL:
http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/document/rdb/RDBDocumentStoreJDBCTest.java?rev=1734279&r1=1734278&r2=1734279&view=diff
==============================================================================
---
jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/document/rdb/RDBDocumentStoreJDBCTest.java
(original)
+++
jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/document/rdb/RDBDocumentStoreJDBCTest.java
Wed Mar 9 15:58:04 2016
@@ -105,9 +105,6 @@ public class RDBDocumentStoreJDBCTest ex
@Test
public void batchFailingInsertResult() throws SQLException {
- // https://issues.apache.org/jira/browse/OAK-3937 and
https://issues.apache.org/jira/browse/OAK-3977
- assumeTrue(super.dsf != DocumentStoreFixture.RDB_PG);
-
String table = ((RDBDocumentStore)
super.ds).getTable(Collection.NODES).getName();
Connection con = super.rdbDataSource.getConnection();
