Skip to content

Commit 2c29d87

Browse files
committed
Fix handling for capitalized colnames in postgres
1 parent 0ac2a3d commit 2c29d87

8 files changed

Lines changed: 12 additions & 8 deletions

File tree

API/pom.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@
3131
<parent>
3232
<groupId>org.ohnlp.backbone</groupId>
3333
<artifactId>backbone-parent</artifactId>
34-
<version>3.0.16</version>
34+
<version>3.0.17</version>
3535
</parent>
3636

3737
<artifactId>api</artifactId>

Core/pom.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
<parent>
88
<groupId>org.ohnlp.backbone</groupId>
99
<artifactId>backbone-parent</artifactId>
10-
<version>3.0.16</version>
10+
<version>3.0.17</version>
1111
</parent>
1212

1313
<artifactId>core</artifactId>

Example-Backbone-Configs/pom.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
<parent>
88
<groupId>org.ohnlp.backbone</groupId>
99
<artifactId>backbone-parent</artifactId>
10-
<version>3.0.16</version>
10+
<version>3.0.17</version>
1111
</parent>
1212

1313
<artifactId>example-backbone-configs</artifactId>

IO/pom.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
<parent>
88
<groupId>org.ohnlp.backbone</groupId>
99
<artifactId>backbone-parent</artifactId>
10-
<version>3.0.16</version>
10+
<version>3.0.17</version>
1111
</parent>
1212

1313
<groupId>org.ohnlp.backbone.io</groupId>

IO/src/main/java/org/ohnlp/backbone/io/jdbc/JDBCExtract.java

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -216,7 +216,11 @@ private String[] findPaginationOrderingColumns(String query) throws ComponentIni
216216
}
217217
if (this.identifierCol != null) {
218218
// User-supplied identifier column exists, make sure it actually exists in query results
219-
if (!colNameToIndex.containsKey(this.identifierCol)) {
219+
String toCheck = this.identifierCol;
220+
if (this.identifierCol.startsWith("\"") && this.identifierCol.endsWith("\"")) {
221+
toCheck = toCheck.substring(1, toCheck.length() - 1);
222+
}
223+
if (!colNameToIndex.containsKey(toCheck)) {
220224
throw new ComponentInitializationException(
221225
new IllegalArgumentException("The supplied identifier_col " + this.identifierCol + " " +
222226
"does not exist in the returned query results. Available columns: " +

Plugin-Manager/pom.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
<parent>
88
<groupId>org.ohnlp.backbone</groupId>
99
<artifactId>backbone-parent</artifactId>
10-
<version>3.0.16</version>
10+
<version>3.0.17</version>
1111
</parent>
1212

1313
<artifactId>plugin-manager</artifactId>

Transforms/pom.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
<parent>
88
<groupId>org.ohnlp.backbone</groupId>
99
<artifactId>backbone-parent</artifactId>
10-
<version>3.0.16</version>
10+
<version>3.0.17</version>
1111
</parent>
1212

1313
<groupId>org.ohnlp.backbone.transforms</groupId>

pom.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66

77
<groupId>org.ohnlp.backbone</groupId>
88
<artifactId>backbone-parent</artifactId>
9-
<version>3.0.16</version>
9+
<version>3.0.17</version>
1010

1111

1212
<properties>

0 commit comments

Comments
 (0)