Skip to content

Commit

Permalink
Merge pull request #4039 from DedunuKarunarathne/stream-close
Browse files Browse the repository at this point in the history
Revert try-with-resources for BufferedReader
  • Loading branch information
DedunuKarunarathne authored Feb 19, 2025
2 parents 36dc5cc + 18e12e3 commit 2e8a7f8
Showing 1 changed file with 8 additions and 12 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -1683,24 +1683,20 @@ private void setClobValue(int queryType, String paramName,
if (value == null) {
sqlQuery.setNull(i + 1, Types.CLOB);
} else {
try (BufferedReader reader = new BufferedReader(new StringReader(value))) {
sqlQuery.setClob(i + 1, reader, value.length());
} catch (IOException e) {
throw new DataServiceFault(e, "Error processing parameter: " + paramName
+ ", Error: " + e.getMessage());
}
// Use of Try-with-resources is removed since it causes "Stream closed" error.
// The JDBC driver will handle closing the stream after reading the data.
sqlQuery.setClob(i + 1, new BufferedReader(new StringReader(value)),
value.length());
}
} else if ("INOUT".equals(paramType)) {
if (value == null) {
((CallableStatement) sqlQuery).setNull(i + 1,
Types.CLOB);
} else {
try (BufferedReader reader = new BufferedReader(new StringReader(value))) {
((CallableStatement) sqlQuery).setClob(i + 1, reader, value.length());
} catch (IOException e) {
throw new DataServiceFault(e, "Error processing parameter: " + paramName + ", Error: "
+ e.getMessage());
}
// Use of Try-with-resources is removed since it causes "Stream closed" error.
// The JDBC driver will handle closing the stream after reading the data.
((CallableStatement) sqlQuery).setClob(i + 1,
new BufferedReader(new StringReader(value)), value.length());
}
((CallableStatement) sqlQuery).registerOutParameter(i + 1,
Types.CLOB);
Expand Down

0 comments on commit 2e8a7f8

Please sign in to comment.