database/mariadb-columnstore-engine
1
0
Fork 0
mirror of https://github.com/mariadb-corporation/mariadb-columnstore-engine.git synced 2025-08-08 14:22:09 +03:00
Code Activity

Merge pull request #700 from mariadb-corporation/MCOL-901_2

Browse Source 
MCOL-901 group_concat() uses only a subset of unique values
This commit is contained in:
Andrew Hutchings
2019-03-06 06:21:42 -05:00
committed by GitHub
parent 7d24f031c7 947eaaef26
commit e5758b794b
2 changed files with 6 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
dbcon/joblist/limitedorderby.cpp
View File

@@ -77,8 +77,6 @@ void LimitedOrderBy::initialize(const RowGroup& rg, const JobInfo& jobInfo)
map<uint32_t, uint32_t>::iterator j = keyToIndexMap.find(i->first);
idbassert(j != keyToIndexMap.end());
// TODO Ordering direction in CSEP differs from
// internal direction representation. This behavior should be fixed
fOrderByCond.push_back(IdbSortSpec(j->second, i->second));
}
@@ -86,16 +84,14 @@ void LimitedOrderBy::initialize(const RowGroup& rg, const JobInfo& jobInfo)
fStart = jobInfo.limitStart;
fCount = jobInfo.limitCount;
// fMemSize = (fStart + fCount) * rg.getRowSize();
IdbOrderBy::initialize(rg);
}
// This must return a proper number of key columns and
// not just a column count.
uint64_t LimitedOrderBy::getKeyLength() const
{
//return (fRow0.getSize() - 2);
return fRow0.getColumnCount();
return fOrderByCond.size();
}