Merge pull request #6518 from taosdata/bug/TD-4715
[TD-4715]bug fix:fix unreset hasRestoreLastColumn flag bug
This commit is contained in:
commit
fe05c1be0f
|
@ -812,6 +812,7 @@ int tsdbRestoreInfo(STsdbRepo *pRepo) {
|
||||||
STable *pTable = pMeta->tables[i];
|
STable *pTable = pMeta->tables[i];
|
||||||
if (pTable == NULL) continue;
|
if (pTable == NULL) continue;
|
||||||
pTable->restoreColumnNum = 0;
|
pTable->restoreColumnNum = 0;
|
||||||
|
pTable->hasRestoreLastColumn = false;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -895,6 +896,7 @@ int tsdbCacheLastData(STsdbRepo *pRepo, STsdbCfg* oldCfg) {
|
||||||
maxTableIdx = i;
|
maxTableIdx = i;
|
||||||
if (cacheLastCol) {
|
if (cacheLastCol) {
|
||||||
pTable->restoreColumnNum = 0;
|
pTable->restoreColumnNum = 0;
|
||||||
|
pTable->hasRestoreLastColumn = false;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -913,6 +915,7 @@ int tsdbCacheLastData(STsdbRepo *pRepo, STsdbCfg* oldCfg) {
|
||||||
}
|
}
|
||||||
if (need_free_last_col) {
|
if (need_free_last_col) {
|
||||||
tsdbFreeLastColumns(pTable);
|
tsdbFreeLastColumns(pTable);
|
||||||
|
pTable->hasRestoreLastColumn = false;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
|
@ -609,6 +609,7 @@ void tsdbFreeLastColumns(STable* pTable) {
|
||||||
pTable->maxColNum = 0;
|
pTable->maxColNum = 0;
|
||||||
pTable->lastColSVersion = -1;
|
pTable->lastColSVersion = -1;
|
||||||
pTable->restoreColumnNum = 0;
|
pTable->restoreColumnNum = 0;
|
||||||
|
pTable->hasRestoreLastColumn = false;
|
||||||
}
|
}
|
||||||
|
|
||||||
int16_t tsdbGetLastColumnsIndexByColId(STable* pTable, int16_t colId) {
|
int16_t tsdbGetLastColumnsIndexByColId(STable* pTable, int16_t colId) {
|
||||||
|
@ -645,6 +646,7 @@ int tsdbInitColIdCacheWithSchema(STable* pTable, STSchema* pSchema) {
|
||||||
pTable->lastColSVersion = schemaVersion(pSchema);
|
pTable->lastColSVersion = schemaVersion(pSchema);
|
||||||
pTable->maxColNum = numOfColumn;
|
pTable->maxColNum = numOfColumn;
|
||||||
pTable->restoreColumnNum = 0;
|
pTable->restoreColumnNum = 0;
|
||||||
|
pTable->hasRestoreLastColumn = false;
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -657,7 +659,7 @@ int tsdbUpdateLastColSchema(STable *pTable, STSchema *pNewSchema) {
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
tsdbInfo("tsdbUpdateLastColSchema:%s,%d->%d", pTable->name->data, pTable->lastColSVersion, schemaVersion(pNewSchema));
|
tsdbDebug("tsdbUpdateLastColSchema:%s,%d->%d", pTable->name->data, pTable->lastColSVersion, schemaVersion(pNewSchema));
|
||||||
|
|
||||||
int16_t numOfCols = pNewSchema->numOfCols;
|
int16_t numOfCols = pNewSchema->numOfCols;
|
||||||
SDataCol *lastCols = (SDataCol*)malloc(numOfCols * sizeof(SDataCol));
|
SDataCol *lastCols = (SDataCol*)malloc(numOfCols * sizeof(SDataCol));
|
||||||
|
@ -802,6 +804,7 @@ static STable *tsdbNewTable() {
|
||||||
pTable->lastCols = NULL;
|
pTable->lastCols = NULL;
|
||||||
pTable->restoreColumnNum = 0;
|
pTable->restoreColumnNum = 0;
|
||||||
pTable->maxColNum = 0;
|
pTable->maxColNum = 0;
|
||||||
|
pTable->hasRestoreLastColumn = false;
|
||||||
pTable->lastColSVersion = -1;
|
pTable->lastColSVersion = -1;
|
||||||
return pTable;
|
return pTable;
|
||||||
}
|
}
|
||||||
|
|
|
@ -2520,7 +2520,7 @@ static bool loadCachedLast(STsdbQueryHandle* pQueryHandle) {
|
||||||
int32_t numOfCols = pTable->maxColNum;
|
int32_t numOfCols = pTable->maxColNum;
|
||||||
|
|
||||||
if (pTable->lastCols == NULL || pTable->maxColNum <= 0) {
|
if (pTable->lastCols == NULL || pTable->maxColNum <= 0) {
|
||||||
tsdbWarn("no last cached for table, uid:%" PRIu64 ",tid:%d", pTable->tableId.uid, pTable->tableId.tid);
|
tsdbWarn("no last cached for table %s, uid:%" PRIu64 ",tid:%d", pTable->name->data, pTable->tableId.uid, pTable->tableId.tid);
|
||||||
continue;
|
continue;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue