Changeset: 27d9c8547be0 for MonetDB
URL: http://dev.monetdb.org/hg/MonetDB?cmd=changeset;node=27d9c8547be0
Modified Files:
sql/src/server/bin_optimizer.c
Branch: default
Log Message:
removed bogus optimization. Fixes bug 2757.
diffs (29 lines):
diff -r 0b3fb80c6d7d -r 27d9c8547be0 sql/src/server/bin_optimizer.c
--- a/sql/src/server/bin_optimizer.c Sat Dec 18 12:53:06 2010 +0100
+++ b/sql/src/server/bin_optimizer.c Sat Dec 18 12:57:47 2010 +0100
@@ -279,25 +279,6 @@
os->optimized = ns->optimized = 3;
return ns;
} else
- /* incase there is no reduction on the left, push
- the topn into the right part */
- if (s->flag != 0 /* topn, ie limit and orderby */ &&
- j->type == st_join &&
- ((isEqJoin(j) &&
- !is_reduced(j) &&
- j->op1->t && j->op1->t == j->op2->h) ||
- j->flag == cmp_all)) {
- stmt *l = j->op1;
- stmt *r = j->op2;
-
- r = stmt_limit(c->sa, r, s->op2, s->op3, s->flag);
- s = stmt_join(c->sa, l, r, cmp_equal);
- ns = _bin_optimizer(c, s);
- assert(os->rewritten==NULL);
- os->rewritten = ns;
- os->optimized = ns->optimized = 3;
- return ns;
- }
/* try to push the limit through the reverse */
if (!s->flag && j->type == st_reverse) {
s = stmt_reverse(c->sa, stmt_limit(c->sa, j->op1,
s->op2,
_______________________________________________
Checkin-list mailing list
[email protected]
http://mail.monetdb.org/mailman/listinfo/checkin-list
