xtern commented on code in PR #2906:
URL: https://github.com/apache/ignite-3/pull/2906#discussion_r1425294308


##########
modules/jdbc/src/main/java/org/apache/ignite/internal/jdbc/JdbcPreparedStatement.java:
##########
@@ -762,7 +762,7 @@ private void executeWithArguments(JdbcStatementType 
statementType) throws SQLExc
         Object[] args = currentArgs == null ? ArrayUtils.OBJECT_EMPTY_ARRAY :
                 
currentArgs.stream().map(this::convertJdbcTypeToInternal).toArray();
 
-        execute0(statementType, sql, args);
+        execute0(statementType, sql, false, args);

Review Comment:
   I don't know where to put this question.
   
   Why don't we allow script execution using PreparedStatement#execute method?
   
   According to the javadoc this should be allowed:
   ```
   The execute method returns a boolean to indicate the form of the first 
result. 
   You must call either the method getResultSet or getUpdateCount to retrieve 
   the result; you must call getMoreResults to move to any subsequent result(s).
   ```



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to