Merge remote-tracking branch 'upstream/branch-0.5.0' into branch-0.5.0-yk

Former-commit-id: 800d8e422e6a1e836389987ad22a6723fe36cadf

cpp/build.sh

@@ -112,13 +112,13 @@ if [[ ${RUN_CPPLINT} == "ON" ]]; then
     fi
     echo "clang-format check passed!"
 
-    # clang-tidy check
-    make check-clang-tidy
-    if [ $? -ne 0 ]; then
-        echo "ERROR! clang-tidy check failed"
-        exit 1
-    fi
-    echo "clang-tidy check passed!"
+#    # clang-tidy check
+#    make check-clang-tidy
+#    if [ $? -ne 0 ]; then
+#        echo "ERROR! clang-tidy check failed"
+#        exit 1
+#    fi
+#    echo "clang-tidy check passed!"
 else
     # compile and build
     make -j 4 || exit 1

cpp/src/metrics/SystemInfo.cpp

@@ -263,8 +263,8 @@ SystemInfo::CPUTemperature() {
                 object += "/temp1_input";
                 FILE *file = fopen(object.c_str(), "r");
                 if (file == nullptr) {
-                    SERVER_LOG_ERROR << "Could not open temperature file"
-                    exit(1);
+                    SERVER_LOG_ERROR << "Could not open temperature file";
+                    return result;
                 }
                 float temp;
                 fscanf(file, "%f", &temp);

cpp/src/utils/ValidationUtil.cpp

@@ -71,7 +71,11 @@ ValidationUtil::ValidateTableName(const std::string& table_name) {
 
 Status
 ValidationUtil::ValidateTableDimension(int64_t dimension) {
-    if (dimension <= 0 || dimension > TABLE_DIMENSION_LIMIT) {
+    if (dimension <= 0) {
+        std::string msg = "Dimension value should be greater than 0";
+        SERVER_LOG_ERROR << msg;
+        return Status(SERVER_INVALID_VECTOR_DIMENSION, msg);
+    } else if (dimension > TABLE_DIMENSION_LIMIT) {
         std::string msg = "Table dimension excceed the limitation: " + std::to_string(TABLE_DIMENSION_LIMIT);
         SERVER_LOG_ERROR << msg;
         return Status(SERVER_INVALID_VECTOR_DIMENSION, msg);