update clang-tidy rules (#3416)

Signed-off-by: yudong.cai <yudong.cai@zilliz.com>

.clang-tidy

@@ -20,7 +20,13 @@ Checks: >
     -*, clang-diagnostic-*, -clang-diagnostic-error,
     clang-analyzer-*, -clang-analyzer-alpha*,
     google-*, -google-runtime-references, -google-readability-todo,
-    modernize-*, -modernize-pass-by-value, -modernize-use-equals-default
+    modernize-*, -modernize-pass-by-value, -modernize-use-equals-default,
+    performance-faster-string-find, performance-for-range-copy,
+    performance-implicit-conversion-in-loop, performance-inefficient-algorithm,
+    performance-trivially-destructible, performance-inefficient-vector-operation,
+    performance-move-const-arg, performance-move-constructor-init,
+    performance-noexcept-move-constructor, performance-no-automatic-move,
+    performance-type-promotion-in-math-fn
 
 # produce HeaderFilterRegex from core/build-support/lint_exclusions.txt with:
 # echo -n '^?!('; sed -e 's/*/\.*/g' core/build-support/lint_exclusions.txt | tr '\n' '|'; echo ')$'

core/src/metrics/SystemInfo.cpp

@@ -370,10 +370,10 @@ SystemInfo::Octets() {
         lastline = line;
     }
     std::vector<size_t> space_position;
-    size_t space_pos = lastline.find(" ");
+    size_t space_pos = lastline.find(' ');
     while (space_pos != std::string::npos) {
         space_position.push_back(space_pos);
-        space_pos = lastline.find(" ", space_pos + 1);
+        space_pos = lastline.find(' ', space_pos + 1);
     }
     // InOctets is between 6th and 7th " " and OutOctets is between 7th and 8th " "
     size_t inoctets_begin = space_position[6] + 1;

core/src/server/grpc_impl/GrpcRequestHandler.cpp

@@ -1323,7 +1323,7 @@ GrpcRequestHandler::Insert(::grpc::ServerContext* context, const ::milvus::grpc:
         auto grpc_float_size = request->fields(i).attr_record().float_value_size();
         auto grpc_double_size = request->fields(i).attr_record().double_value_size();
         const auto& field = request->fields(i);
-        auto field_name = field.field_name();
+        auto& field_name = field.field_name();
 
         std::vector<uint8_t> temp_data;
         if (grpc_int32_size > 0) {

core/src/utils/Status.cpp

@@ -38,7 +38,7 @@ Status::Status(const Status& s) {
     CopyFrom(s);
 }
 
-Status::Status(Status&& s) {
+Status::Status(Status&& s) noexcept {
     MoveFrom(s);
 }
 
@@ -49,7 +49,7 @@ Status::operator=(const Status& s) {
 }
 
 Status&
-Status::operator=(Status&& s) {
+Status::operator=(Status&& s) noexcept {
     MoveFrom(s);
     return *this;
 }

core/src/utils/Status.h

@@ -36,13 +36,13 @@ class Status {
 
     Status(const Status& s);
 
-    Status(Status&& s);
+    Status(Status&& s) noexcept;
 
     Status&
     operator=(const Status& s);
 
     Status&
-    operator=(Status&& s);
+    operator=(Status&& s) noexcept;
 
     static Status
     OK() {