database/mariadb-columnstore-engine
1
0
Fork 0
mirror of https://github.com/mariadb-corporation/mariadb-columnstore-engine.git synced 2025-07-30 19:23:07 +03:00
Code Activity

Merge pull request #1367 from dhall-MariaDB/MCOL-3827

Browse Source 
MCOL-3827 Optimize out sort on SubQuery in Select
This commit is contained in:
Gagan Goel
2020-08-18 14:54:36 -04:00
committed by GitHub
parent 59cae761d5 e7b8abfdb9
commit f1759e6560
2 changed files with 19 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
dbcon/joblist/jlf_subquery.cpp
View File

@ -823,7 +823,7 @@ void addOrderByAndLimit(CalpontSelectExecutionPlan* csep, JobInfo& jobInfo)
if (sc == NULL) if (sc == NULL)
{ {
const ReturnedColumn* rc = dynamic_cast<const ReturnedColumn*>(orderByCols[i].get()); const ReturnedColumn* rc = dynamic_cast<const ReturnedColumn*>(orderByCols[i].get());
uint64_t eid = rc->expressionId(); uint32_t eid = rc->expressionId();
CalpontSystemCatalog::ColType ct = rc->resultType(); CalpontSystemCatalog::ColType ct = rc->resultType();
tupleKey = getExpTupleKey(jobInfo, eid); tupleKey = getExpTupleKey(jobInfo, eid);
jobInfo.orderByColVec.push_back(make_pair(tupleKey, orderByCols[i]->asc())); jobInfo.orderByColVec.push_back(make_pair(tupleKey, orderByCols[i]->asc()));

28
dbcon/mysql/ha_mcs_execplan.cpp
View File

@ -7791,12 +7791,16 @@ int getSelectPlan(gp_walk_info& gwi, SELECT_LEX& select_lex,
} }
} }
if (ordercol->direction == ORDER::ORDER_ASC) SimpleColumn* sc = dynamic_cast<SimpleColumn*>(rc);
rc->asc(true); if (!sc || sc->oid())
else {
rc->asc(false); if (ordercol->direction == ORDER::ORDER_ASC)
rc->asc(true);
else
rc->asc(false);
gwi.orderByCols.push_back(SRCP(rc)); gwi.orderByCols.push_back(SRCP(rc));
}
} }
} }
// make sure columnmap, returnedcols and count(*) arg_list are not empty // make sure columnmap, returnedcols and count(*) arg_list are not empty
@ -9583,12 +9587,16 @@ int getGroupPlan(gp_walk_info& gwi, SELECT_LEX& select_lex, SCSEP& csep, cal_gro
} }
} }
if (ordercol->direction == ORDER::ORDER_ASC) SimpleColumn* sc = dynamic_cast<SimpleColumn*>(rc);
rc->asc(true); if (!sc || sc->oid())
else {
rc->asc(false); if (ordercol->direction == ORDER::ORDER_ASC)
rc->asc(true);
else
rc->asc(false);
gwi.orderByCols.push_back(SRCP(rc)); gwi.orderByCols.push_back(SRCP(rc));
}
} }