[GitHub] [ignite-3] AMashenkov commented on a diff in pull request #843: IGNITE-16963 SQL API: Add batched DML queries support

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

AMashenkov commented on code in PR #843:
URL: https://github.com/apache/ignite-3/pull/843#discussion_r887695077


##########
modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/api/SessionImpl.java:
##########
@@ -309,7 +323,11 @@ public static <T> T await(CompletionStage<T> stage) {
         try {
             return stage.toCompletableFuture().get();
         } catch (Throwable e) {
-            throw new IgniteException(e);
+            if (e instanceof ExecutionException) {
+                throw new IgniteException(e.getCause());
+            } else {
+                throw new IgniteException(e);
+            }

Review Comment:
   ```suggestion
               catch (ExecutionException e) {
                   throw new IgniteException(e.getCause());
               }
               catch (Throwable e) {
                   throw new IgniteException(e);
               }
   ```



-- 
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