Skip to content

Commit

Permalink
Merge pull request #2050 from eclipse/GH-2048-remove-commented-code
Browse files Browse the repository at this point in the history
GH-2048 remove code that has been commented out
  • Loading branch information
hmottestad authored Mar 29, 2020
2 parents ff495e4 + 487f238 commit 37cb752
Showing 1 changed file with 6 additions and 13 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@

/**
* A query optimizer that re-orders nested Joins.
*
*
* @author Arjohn Kampman
* @author James Leigh
*/
Expand All @@ -48,7 +48,7 @@ public QueryJoinOptimizer(EvaluationStatistics statistics) {

/**
* Applies generally applicable optimizations: path expressions are sorted from more to less specific.
*
*
* @param tupleExpr
*/
@Override
Expand Down Expand Up @@ -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
*/
Expand Down Expand Up @@ -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;
}

Expand Down

0 comments on commit 37cb752

Please sign in to comment.