opt performance by removing the mmap operation

This commit is contained in:
hjxilinx 2019-12-19 13:33:12 +08:00
parent 24d93f5729
commit 897c23bd9a
1 changed files with 19 additions and 19 deletions

View File

@ -372,24 +372,24 @@ static int32_t doOpenQueryFileData(SQInfo *pQInfo, SQueryFilesInfo *pVnodeFileIn
return -1; return -1;
} }
pVnodeFileInfo->pHeaderFileData = // pVnodeFileInfo->pHeaderFileData =
mmap(NULL, pVnodeFileInfo->headFileSize, PROT_READ, MAP_SHARED, pVnodeFileInfo->headerFd, 0); // mmap(NULL, pVnodeFileInfo->headFileSize, PROT_READ, MAP_SHARED, pVnodeFileInfo->headerFd, 0);
//
if (pVnodeFileInfo->pHeaderFileData == MAP_FAILED) { // if (pVnodeFileInfo->pHeaderFileData == MAP_FAILED) {
pVnodeFileInfo->pHeaderFileData = NULL; // pVnodeFileInfo->pHeaderFileData = NULL;
//
doCloseQueryFileInfoFD(pVnodeFileInfo); // doCloseQueryFileInfoFD(pVnodeFileInfo);
doInitQueryFileInfoFD(pVnodeFileInfo); // doInitQueryFileInfoFD(pVnodeFileInfo);
//
dError("QInfo:%p failed to mmap header file:%s, size:%lld, %s", pQInfo, pVnodeFileInfo->headerFilePath, // dError("QInfo:%p failed to mmap header file:%s, size:%lld, %s", pQInfo, pVnodeFileInfo->headerFilePath,
pVnodeFileInfo->headFileSize, strerror(errno)); // pVnodeFileInfo->headFileSize, strerror(errno));
//
return -1; // return -1;
} else { // } else {
if (madvise(pVnodeFileInfo->pHeaderFileData, pVnodeFileInfo->headFileSize, MADV_SEQUENTIAL) == -1) { // if (madvise(pVnodeFileInfo->pHeaderFileData, pVnodeFileInfo->headFileSize, MADV_SEQUENTIAL) == -1) {
dError("QInfo:%p failed to advise kernel the usage of header file, reason:%s", pQInfo, strerror(errno)); // dError("QInfo:%p failed to advise kernel the usage of header file, reason:%s", pQInfo, strerror(errno));
} // }
} // }
return TSDB_CODE_SUCCESS; return TSDB_CODE_SUCCESS;
} }
@ -450,7 +450,7 @@ char *vnodeGetHeaderFileData(SQueryRuntimeEnv *pRuntimeEnv, int32_t vnodeId, int
} }
} }
return pVnodeFileInfo->pHeaderFileData; return 1;//pVnodeFileInfo->pHeaderFileData;
} }
/* /*