more comments

line · Jul 16, 2024 · caa6459 · caa6459
1 parent be26926
commit caa6459
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/xds/src/main/java/com/linecorp/armeria/xds/client/endpoint/LocalityRoutingStateFactory.java b/xds/src/main/java/com/linecorp/armeria/xds/client/endpoint/LocalityRoutingStateFactory.java
@@ -68,7 +68,10 @@ LocalityRoutingState create(PrioritySet upstreamPrioritySet, PrioritySet localPr
         final long localPercentToRoute = localLocalityPercentages != null ?
                                          localLocalityPercentages.localPercentToRoute() : 0;
         final List<Locality> upstreamLocalities =
-                upstreamHealthyPerLocality.keySet().stream().filter(locality -> !locality.equals(localLocality))
+                upstreamHealthyPerLocality.keySet().stream()
+                                          .filter(locality -> !locality.equals(localLocality))
+                                          // we assume there aren't many localities anyway, so
+                                          // we just simply sort by the string-ed value
                                           .sorted(Comparator.comparing(AbstractMessage::toString))
                                           .collect(Collectors.toList());
         final ImmutableList.Builder<ResidualCapacity> residualCapacity = ImmutableList.builder();