Skip to content

Commit

Permalink
Fix for #48: Switched to the filter placement optimizer intended for …
Browse files Browse the repository at this point in the history
…left joins - although not exactly sure anymore what the code really does
  • Loading branch information
Aklakan committed May 15, 2014
1 parent 7805e75 commit 2b14dc3
Showing 1 changed file with 16 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -112,7 +112,23 @@ public static Op _optimize(OpTopN op, RestrictionManagerImpl cnf) {
}



public static Op _optimize(OpJoin op, RestrictionManagerImpl cnf) {
Factory2<Op> factory = new Factory2<Op>() {
@Override
public Op create(Op a, Op b) {
Op result = OpJoin.create(a, b);
return result;
}

};

Op result = handleLeftJoin(op.getLeft(), op.getRight(), cnf, factory);
return result;

}

public static Op _optimizeBreaking(OpJoin op, RestrictionManagerImpl cnf) {

RestrictionManagerImpl leftCnf = filterByVars(cnf, op.getLeft());
RestrictionManagerImpl rightCnf = filterByVars(cnf, op.getRight());
Expand Down

0 comments on commit 2b14dc3

Please sign in to comment.