Remove pareto-optimality check of via connections

The check had really bad performance and non-pareto optimal results don't cause that much problems

Author: optionsome

raptor/src/main/java/org/opentripplanner/raptor/api/request/RaptorViaLocation.java

@@ -138,24 +138,10 @@ private List<RaptorViaConnection> validateConnections(List<BuilderStopAndTransfe
     if (connections.isEmpty()) {
       throw new IllegalArgumentException("At least one connection is required.");
     }
-    var list = connections
+    return connections
       .stream()
       .map(it -> RaptorViaConnection.of(this, it.fromStop, it.transfer))
       .toList();
-
-    // Compare all pairs to check for duplicates and non-optimal connections
-    for (int i = 0; i < list.size(); ++i) {
-      var a = list.get(i);
-      for (int j = i + 1; j < list.size(); ++j) {
-        var b = list.get(j);
-        if (a.isBetterOrEqual(b) || b.isBetterOrEqual(a)) {
-          throw new IllegalArgumentException(
-            "All connection need to be pareto-optimal: (" + a + ") <-> (" + b + ")"
-          );
-        }
-      }
-    }
-    return list;
   }
 
   public abstract static sealed class AbstractBuilder<T extends AbstractBuilder> {