fix bugs
This commit is contained in:
parent
b342561535
commit
8fe95c982c
|
@ -68,6 +68,7 @@ typedef struct STopBotResItem {
|
||||||
typedef struct STopBotRes {
|
typedef struct STopBotRes {
|
||||||
int32_t maxSize;
|
int32_t maxSize;
|
||||||
int16_t type; //store the original input type, used in merge function
|
int16_t type; //store the original input type, used in merge function
|
||||||
|
int32_t numOfItems;
|
||||||
STopBotResItem* pItems;
|
STopBotResItem* pItems;
|
||||||
} STopBotRes;
|
} STopBotRes;
|
||||||
|
|
||||||
|
@ -2718,12 +2719,13 @@ int32_t topFunction(SqlFunctionCtx* pCtx) {
|
||||||
}
|
}
|
||||||
|
|
||||||
static void topTransferInfo(SqlFunctionCtx* pCtx, STopBotRes* pInput) {
|
static void topTransferInfo(SqlFunctionCtx* pCtx, STopBotRes* pInput) {
|
||||||
for (int32_t i = 0; i < pInput->maxSize; i++) {
|
for (int32_t i = 0; i < pInput->numOfItems; i++) {
|
||||||
addResult(pCtx, &pInput->pItems[i], pInput->type, true);
|
addResult(pCtx, &pInput->pItems[i], pInput->type, true);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
int32_t topFunctionMerge(SqlFunctionCtx* pCtx) {
|
int32_t topFunctionMerge(SqlFunctionCtx* pCtx) {
|
||||||
|
SResultRowEntryInfo* pEntryInfo = GET_RES_INFO(pCtx);
|
||||||
SInputColumnInfoData* pInput = &pCtx->input;
|
SInputColumnInfoData* pInput = &pCtx->input;
|
||||||
SColumnInfoData* pCol = pInput->pData[0];
|
SColumnInfoData* pCol = pInput->pData[0];
|
||||||
ASSERT(pCol->info.type == TSDB_DATA_TYPE_BINARY);
|
ASSERT(pCol->info.type == TSDB_DATA_TYPE_BINARY);
|
||||||
|
@ -2736,7 +2738,7 @@ int32_t topFunctionMerge(SqlFunctionCtx* pCtx) {
|
||||||
pInfo->maxSize = pInputInfo->maxSize;
|
pInfo->maxSize = pInputInfo->maxSize;
|
||||||
pInfo->type = pInputInfo->type;
|
pInfo->type = pInputInfo->type;
|
||||||
topTransferInfo(pCtx, pInputInfo);
|
topTransferInfo(pCtx, pInputInfo);
|
||||||
SET_VAL(GET_RES_INFO(pCtx), pInputInfo->maxSize, pInputInfo->maxSize);
|
SET_VAL(GET_RES_INFO(pCtx), pEntryInfo->numOfRes, pEntryInfo->numOfRes);
|
||||||
|
|
||||||
return TSDB_CODE_SUCCESS;
|
return TSDB_CODE_SUCCESS;
|
||||||
}
|
}
|
||||||
|
@ -2812,6 +2814,8 @@ void doAddIntoResult(SqlFunctionCtx* pCtx, void* pData, int32_t rowIndex, SSData
|
||||||
|
|
||||||
// allocate the buffer and keep the data of this row into the new allocated buffer
|
// allocate the buffer and keep the data of this row into the new allocated buffer
|
||||||
pEntryInfo->numOfRes++;
|
pEntryInfo->numOfRes++;
|
||||||
|
// accumulate number of items for each vgroup, this info is needed for merge
|
||||||
|
pRes->numOfItems++;
|
||||||
taosheapsort((void*)pItems, sizeof(STopBotResItem), pEntryInfo->numOfRes, (const void*)&type, topBotResComparFn,
|
taosheapsort((void*)pItems, sizeof(STopBotResItem), pEntryInfo->numOfRes, (const void*)&type, topBotResComparFn,
|
||||||
!isTopQuery);
|
!isTopQuery);
|
||||||
} else { // replace the minimum value in the result
|
} else { // replace the minimum value in the result
|
||||||
|
@ -2983,6 +2987,8 @@ void addResult(SqlFunctionCtx* pCtx, STopBotResItem* pSourceItem, int16_t type,
|
||||||
pItem->tuplePos.pageId = -1;
|
pItem->tuplePos.pageId = -1;
|
||||||
replaceTupleData(&pItem->tuplePos, &pSourceItem->tuplePos);
|
replaceTupleData(&pItem->tuplePos, &pSourceItem->tuplePos);
|
||||||
pEntryInfo->numOfRes++;
|
pEntryInfo->numOfRes++;
|
||||||
|
// accumulate number of items for each vgroup, this info is needed for merge
|
||||||
|
pRes->numOfItems++;
|
||||||
taosheapsort((void*)pItems, sizeof(STopBotResItem), pEntryInfo->numOfRes, (const void*)&type, topBotResComparFn,
|
taosheapsort((void*)pItems, sizeof(STopBotResItem), pEntryInfo->numOfRes, (const void*)&type, topBotResComparFn,
|
||||||
!isTopQuery);
|
!isTopQuery);
|
||||||
} else { // replace the minimum value in the result
|
} else { // replace the minimum value in the result
|
||||||
|
|
Loading…
Reference in New Issue