From 487f238aa9f2ea29d1a79b03d4117aad4e453274 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ha=CC=8Avard=20Ottestad?= <hmottestad@gmail.com>
Date: Sun, 29 Mar 2020 17:05:00 +0200
Subject: [PATCH] #2048 remove code that has been commented out since this code
 was initially written
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Signed-off-by: Håvard Ottestad <hmottestad@gmail.com>
---
 .../evaluation/impl/QueryJoinOptimizer.java   | 19 ++++++-------------
 1 file changed, 6 insertions(+), 13 deletions(-)

diff --git a/core/queryalgebra/evaluation/src/main/java/org/eclipse/rdf4j/query/algebra/evaluation/impl/QueryJoinOptimizer.java b/core/queryalgebra/evaluation/src/main/java/org/eclipse/rdf4j/query/algebra/evaluation/impl/QueryJoinOptimizer.java
index f7882826a9b..93d6d35dd6b 100644
--- a/core/queryalgebra/evaluation/src/main/java/org/eclipse/rdf4j/query/algebra/evaluation/impl/QueryJoinOptimizer.java
+++ b/core/queryalgebra/evaluation/src/main/java/org/eclipse/rdf4j/query/algebra/evaluation/impl/QueryJoinOptimizer.java
@@ -30,7 +30,7 @@
 
 /**
  * A query optimizer that re-orders nested Joins.
- * 
+ *
  * @author Arjohn Kampman
  * @author James Leigh
  */
@@ -48,7 +48,7 @@ public QueryJoinOptimizer(EvaluationStatistics statistics) {
 
 	/**
 	 * Applies generally applicable optimizations: path expressions are sorted from more to less specific.
-	 * 
+	 *
 	 * @param tupleExpr
 	 */
 	@Override
@@ -231,17 +231,17 @@ protected List<TupleExpr> getSubSelects(List<TupleExpr> expressions) {
 		 * maximized.
 		 * <p>
 		 * Example: reordering
-		 * 
+		 *
 		 * <pre>
 		 *   [f] [a b c] [e f] [a d] [b e]
 		 * </pre>
-		 * 
+		 *
 		 * should result in:
-		 * 
+		 *
 		 * <pre>
 		 *   [a b c] [a d] [b e] [e f] [f]
 		 * </pre>
-		 * 
+		 *
 		 * @param subselects the original ordering of expressions
 		 * @return the optimized ordering of expressions
 		 */
@@ -421,13 +421,6 @@ protected double getTupleExprCardinality(TupleExpr tupleExpr, Map<TupleExpr, Dou
 				}
 			}
 
-			// Prefer patterns that bind more variables
-			// List<Var> distinctUnboundVars = getUnboundVars(new
-			// HashSet<Var>(vars));
-			// if (distinctUnboundVars.size() >= 2) {
-			// cardinality /= distinctUnboundVars.size();
-			// }
-
 			return cardinality;
 		}