zstan commented on a change in pull request #9751:
URL: https://github.com/apache/ignite/pull/9751#discussion_r794227978
##########
File path:
modules/calcite/src/main/java/org/apache/ignite/internal/processors/query/calcite/RootQuery.java
##########
@@ -65,10 +67,10 @@
/** Parameters. */
private final Object[] params;
- /** remote nodes */
- private final Set<UUID> remotes;
+ /** Remote nodes unfinished fragments count. */
+ private final Map<UUID, AtomicInteger> remoteFragments;
Review comment:
I suppose that simple Integer would be enough here, possible conc part
defended be mux monitor, i recheck with Integer and all tests is ok, did i miss
smth ?
--
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]
