Commit 61864021 authored by Sergey Petrunya's avatar Sergey Petrunya

Code cleanup, remove junk comments

parent 3f595889
...@@ -727,9 +727,6 @@ Item_singlerow_subselect::select_transformer(JOIN *join) ...@@ -727,9 +727,6 @@ Item_singlerow_subselect::select_transformer(JOIN *join)
void Item_singlerow_subselect::store(uint i, Item *item) void Item_singlerow_subselect::store(uint i, Item *item)
{ {
row[i]->store(item); row[i]->store(item);
//psergey-merge: can do without that: row[i]->cache_value();
//psergey-backport-timours: ^ really, without that ^
//psergey-try-merge-again:
row[i]->cache_value(); row[i]->cache_value();
} }
......
...@@ -1159,7 +1159,7 @@ static bool convert_subq_to_jtbm(JOIN *parent_join, ...@@ -1159,7 +1159,7 @@ static bool convert_subq_to_jtbm(JOIN *parent_join,
parent_join->select_lex->where= parent_join->conds; parent_join->select_lex->where= parent_join->conds;
} }
/* Don't unlink the child, the subquery is still there and used */ /* Don't unlink the child subselect, as the subquery will be used. */
DBUG_RETURN(FALSE); DBUG_RETURN(FALSE);
} }
......
...@@ -4743,18 +4743,6 @@ best_access_path(JOIN *join, ...@@ -4743,18 +4743,6 @@ best_access_path(JOIN *join,
} }
else else
{ {
#if 0
/* Estimate cost of reading table. */
if (jtbm_subselect) //psergey-jtbm-todo: why the difference?
tmp= s->read_time;
else
tmp= s->table->file->scan_time();
//psergey-debug:
if (!jtbm_subselect && fabs(s->read_time - s->table->file->scan_time()) > 1.0)
{
fprintf(stderr, "Q:%s\n", thd->query());
}
#endif
tmp= s->read_time; tmp= s->read_time;
if ((s->table->map & join->outer_join) || disable_jbuf) // Can't use join cache if ((s->table->map & join->outer_join) || disable_jbuf) // Can't use join cache
{ {
...@@ -9909,16 +9897,10 @@ static uint reset_nj_counters(JOIN *join, List<TABLE_LIST> *join_list) ...@@ -9909,16 +9897,10 @@ static uint reset_nj_counters(JOIN *join, List<TABLE_LIST> *join_list)
if ((nested_join= table->nested_join)) if ((nested_join= table->nested_join))
{ {
nested_join->counter= 0; nested_join->counter= 0;
//nested_join->n_tables= my_count_bits(nested_join->used_tables &
// ~join->eliminated_tables);
nested_join->n_tables= reset_nj_counters(join, &nested_join->join_list); nested_join->n_tables= reset_nj_counters(join, &nested_join->join_list);
if (!nested_join->n_tables) if (!nested_join->n_tables)
is_eliminated_nest= TRUE; is_eliminated_nest= TRUE;
} }
//if (!table->table || (table->table->map & ~join->eliminated_tables))
//psergey-merge10^
//if (!table->table && (table->table->map & ~join->eliminated_tables))
//psergey-merge11^
if ((!table->table && !is_eliminated_nest) || if ((!table->table && !is_eliminated_nest) ||
(table->table && (table->table->map & ~join->eliminated_tables))) (table->table && (table->table->map & ~join->eliminated_tables)))
n++; n++;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment