Merge pull request #17156 from taosdata/fix/TD-19273
fix(query): fix tbname mismatch with multiple rows output functions
This commit is contained in:
commit
3bea98d2ac
|
@ -1273,6 +1273,14 @@ static void doCopyResultToDataBlock(SExprInfo* pExprInfo, int32_t numOfExprs, SR
|
||||||
|
|
||||||
pCtx[j].resultInfo = getResultEntryInfo(pRow, j, rowEntryOffset);
|
pCtx[j].resultInfo = getResultEntryInfo(pRow, j, rowEntryOffset);
|
||||||
if (pCtx[j].fpSet.finalize) {
|
if (pCtx[j].fpSet.finalize) {
|
||||||
|
if (strcmp(pCtx[j].pExpr->pExpr->_function.functionName, "_group_key") == 0) {
|
||||||
|
// for groupkey along with functions that output multiple lines(e.g. Histogram)
|
||||||
|
// need to match groupkey result for each output row of that function.
|
||||||
|
if (pCtx[j].resultInfo->numOfRes != 0) {
|
||||||
|
pCtx[j].resultInfo->numOfRes = pRow->numOfRows;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
int32_t code = pCtx[j].fpSet.finalize(&pCtx[j], pBlock);
|
int32_t code = pCtx[j].fpSet.finalize(&pCtx[j], pBlock);
|
||||||
if (TAOS_FAILED(code)) {
|
if (TAOS_FAILED(code)) {
|
||||||
qError("%s build result data block error, code %s", GET_TASKID(pTaskInfo), tstrerror(code));
|
qError("%s build result data block error, code %s", GET_TASKID(pTaskInfo), tstrerror(code));
|
||||||
|
|
|
@ -6157,7 +6157,10 @@ int32_t groupKeyFinalize(SqlFunctionCtx* pCtx, SSDataBlock* pBlock) {
|
||||||
SGroupKeyInfo* pInfo = GET_ROWCELL_INTERBUF(pResInfo);
|
SGroupKeyInfo* pInfo = GET_ROWCELL_INTERBUF(pResInfo);
|
||||||
|
|
||||||
if (pInfo->hasResult) {
|
if (pInfo->hasResult) {
|
||||||
colDataAppend(pCol, pBlock->info.rows, pInfo->data, pInfo->isNull ? true : false);
|
int32_t currentRow = pBlock->info.rows;
|
||||||
|
for (; currentRow < pBlock->info.rows + pResInfo->numOfRes; ++currentRow) {
|
||||||
|
colDataAppend(pCol, currentRow, pInfo->data, pInfo->isNull ? true : false);
|
||||||
|
}
|
||||||
} else {
|
} else {
|
||||||
pResInfo->numOfRes = 0;
|
pResInfo->numOfRes = 0;
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue