sanpwc commented on code in PR #4394:
URL: https://github.com/apache/ignite-3/pull/4394#discussion_r1763292921
##########
modules/partition-distribution/src/main/java/org/apache/ignite/internal/partitiondistribution/PartitionDistributionUtils.java:
##########
@@ -27,19 +27,19 @@
import java.util.Set;
/**
- * Stateless affinity utils that produces helper methods for an affinity
assignments calculation.
+ * Stateless distribution utils that produces helper methods for an
assignments distribution calculation.
*/
-public class AffinityUtils {
+public class PartitionDistributionUtils {
/**
- * Calculates affinity assignments.
+ * Calculates assignments distribution.
*
* @param dataNodes Data nodes.
* @param partitions Partitions count.
* @param replicas Replicas count.
* @return List assignments by partition.
*/
public static List<Set<Assignment>>
calculateAssignments(Collection<String> dataNodes, int partitions, int
replicas) {
- List<Set<String>> affinityNodes =
RendezvousAffinityFunction.assignPartitions(
+ List<Set<String>> nodes =
RendezvousDistributionFunction.assignPartitions(
Review Comment:
Should we use assignments here instead of nodes?
--
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
