Revert try-with-resources for BufferedReader

Reverting try-with-resources for BufferedReader since it gets closed before the query executes, causing a Stream closed error. Fixes: wso2#3985
DedunuKarunarathne · Feb 19, 2025 · 18e12e3 · 18e12e3
1 parent 36dc5cc
commit 18e12e3
Showing 1 changed file with 8 additions and 12 deletions.
diff --git a/...src/main/java/org/wso2/micro/integrator/dataservices/core/description/query/SQLQuery.java b/...src/main/java/org/wso2/micro/integrator/dataservices/core/description/query/SQLQuery.java
@@ -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);