From 897c23bd9a4abbf52a43c0d5009054f452056653 Mon Sep 17 00:00:00 2001
From: hjxilinx <hjxilinx@163.com>
Date: Thu, 19 Dec 2019 13:33:12 +0800
Subject: [PATCH] opt performance by removing the mmap operation

---
 src/system/detail/src/vnodeQueryImpl.c | 38 +++++++++++++-------------
 1 file changed, 19 insertions(+), 19 deletions(-)

diff --git a/src/system/detail/src/vnodeQueryImpl.c b/src/system/detail/src/vnodeQueryImpl.c
index c7e044eb6c..a34a7bd06a 100644
--- a/src/system/detail/src/vnodeQueryImpl.c
+++ b/src/system/detail/src/vnodeQueryImpl.c
@@ -372,24 +372,24 @@ static int32_t doOpenQueryFileData(SQInfo *pQInfo, SQueryFilesInfo *pVnodeFileIn
     return -1;
   }
 
-  pVnodeFileInfo->pHeaderFileData =
-      mmap(NULL, pVnodeFileInfo->headFileSize, PROT_READ, MAP_SHARED, pVnodeFileInfo->headerFd, 0);
-
-  if (pVnodeFileInfo->pHeaderFileData == MAP_FAILED) {
-    pVnodeFileInfo->pHeaderFileData = NULL;
-
-    doCloseQueryFileInfoFD(pVnodeFileInfo);
-    doInitQueryFileInfoFD(pVnodeFileInfo);
-
-    dError("QInfo:%p failed to mmap header file:%s, size:%lld, %s", pQInfo, pVnodeFileInfo->headerFilePath,
-           pVnodeFileInfo->headFileSize, strerror(errno));
-
-    return -1;
-  } else {
-    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));
-    }
-  }
+//  pVnodeFileInfo->pHeaderFileData =
+//      mmap(NULL, pVnodeFileInfo->headFileSize, PROT_READ, MAP_SHARED, pVnodeFileInfo->headerFd, 0);
+//
+//  if (pVnodeFileInfo->pHeaderFileData == MAP_FAILED) {
+//    pVnodeFileInfo->pHeaderFileData = NULL;
+//
+//    doCloseQueryFileInfoFD(pVnodeFileInfo);
+//    doInitQueryFileInfoFD(pVnodeFileInfo);
+//
+//    dError("QInfo:%p failed to mmap header file:%s, size:%lld, %s", pQInfo, pVnodeFileInfo->headerFilePath,
+//           pVnodeFileInfo->headFileSize, strerror(errno));
+//
+//    return -1;
+//  } else {
+//    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));
+//    }
+//  }
 
   return TSDB_CODE_SUCCESS;
 }
@@ -450,7 +450,7 @@ char *vnodeGetHeaderFileData(SQueryRuntimeEnv *pRuntimeEnv, int32_t vnodeId, int
     }
   }
 
-  return pVnodeFileInfo->pHeaderFileData;
+  return 1;//pVnodeFileInfo->pHeaderFileData;
 }
 
 /*
-- 
GitLab