xtern commented on code in PR #2789:
URL: https://github.com/apache/ignite-3/pull/2789#discussion_r1390723760
##########
modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/QueryProcessor.java:
##########
@@ -53,4 +54,33 @@ CompletableFuture<AsyncSqlCursor<List<Object>>>
querySingleAsync(
String qry,
Object... params
);
+
+ /**
+ * Execute the multi-statement query with given schema name and parameters.
+ *
+ * @param properties User query properties. See {@link QueryProperty} for
available properties.
+ * @param transactions Transactions facade.
+ * @param transaction A transaction to use for query execution. If null,
an implicit transaction
+ * will be started by provided transactions facade.
+ * @param qry Single statement SQL query.
+ * @param params Query parameters.
+ * @return Sql cursor.
+ *
+ * @throws IgniteException in case of an error.
+ */
+ CompletableFuture<AsyncSqlCursorIterator<List<Object>>> queryScriptAsync(
Review Comment:
Seems current() with prefetch adds complications for the user.
We do not have a method for checking that the iterator is empty (hasRowSet),
so the user must, after receiving the iterator, not only call hasNext/fetchNext
but also check that "current" exists or not.
--
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]
