You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@ignite.apache.org by GitBox <gi...@apache.org> on 2022/02/01 11:40:13 UTC

[GitHub] [ignite] alex-plekhanov commented on a change in pull request #9751: IGNITE-16315 Execution workflow optimization

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: notifications-unsubscribe@ignite.apache.org

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