[GitHub] [ignite-3] AMashenkov commented on a change in pull request #639: IGNITE-16201 JDBC: Batch execution

[GitBox <gi...@apache.org>]

AMashenkov commented on a change in pull request #639:
URL: https://github.com/apache/ignite-3/pull/639#discussion_r802428257



##########
File path: modules/client-common/src/main/java/org/apache/ignite/client/proto/query/event/BatchExecuteResult.java
##########
@@ -30,12 +30,30 @@
     /** Update counts. */
     private int[] updateCnts;
 
+    /** Error code. */
+    private int errorCode;
+
     /**
      * Constructor.
      */
     public BatchExecuteResult() {
     }
 
+    /**
+     * Constructor.
+     *
+     * @param status Status code.
+     * @param errorCode Product error code.
+     * @param errorMessage Error message.
+     * @param updateCnts Array with success and error markers.
+     */
+    public BatchExecuteResult(int status, int errorCode, String errorMessage, int[] updateCnts) {

Review comment:
       The constructor looks ambiguous. 
   Is it possible to get a error and result at the same time?




-- 
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: notifications-unsubscribe@ignite.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org