alex-plekhanov commented on a change in pull request #9751:
URL: https://github.com/apache/ignite/pull/9751#discussion_r796510600
##########
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:
Yes, a simple Integer is enough, but Integer is a boxed type, a new
instance of Integer should be created on each increment. `AtomicInteger` here
used as int holder. It can be also something like `int[]`, but as for me,
atomic integer is more readable.
--
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]
