vldpyatkov commented on code in PR #2537:
URL: https://github.com/apache/ignite-3/pull/2537#discussion_r1837152689
##########
modules/client-handler/src/main/java/org/apache/ignite/client/handler/requests/table/ClientTupleGetAllRequest.java:
##########
@@ -49,6 +49,7 @@ public static CompletableFuture<Void> process(
ClientResourceRegistry resources
) {
return readTableAsync(in, tables).thenCompose(table -> {
+ // TODO: IGNITE-23603 We have to create an implicit transaction,
but leave a possibility to start RO direct.
Review Comment:
I thought again, and currently we still have this issue only in batch get
operation.
We have to determine which transaction has to be started, RO or RW. It
depends on if all keys are in one partition 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: notifications-unsubscr...@ignite.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
