ygerzhedovich commented on code in PR #4613:
URL: https://github.com/apache/ignite-3/pull/4613#discussion_r1810486439


##########
modules/sql-engine/src/integrationTest/java/org/apache/ignite/internal/sql/api/ItSqlApiBaseTest.java:
##########
@@ -1007,14 +1007,14 @@ protected ResultSet<SqlRow> executeForRead(IgniteSql 
sql, @Nullable Transaction
 
     protected abstract ResultSet<SqlRow> executeForRead(IgniteSql sql, 
@Nullable Transaction tx, Statement statement, Object... args);
 
-    protected SqlException checkSqlError(
+    protected void checkSqlError(

Review Comment:
   amended due to many warning fro not used return value in the project



-- 
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-unsubscr...@ignite.apache.org

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

Reply via email to