skip unkown types in flt

oceanbase · Sep 8, 2023 · dd9169a · dd9169a
1 parent d4e5040
commit dd9169a
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 23 deletions.
diff --git a/src/sql/monitor/flt/ob_flt_extra_info.cpp b/src/sql/monitor/flt/ob_flt_extra_info.cpp
@@ -158,9 +158,7 @@ int FLTControlInfo::deserialize_field(FullLinkTraceExtraInfoId extra_id, const i
       break;
     }
     default: {
-      ret = OB_NOT_SUPPORTED;
-      LOG_USER_ERROR(OB_NOT_SUPPORTED, "this extra info id");
-      OB_LOG(WARN,"invalid extra info id", K(extra_id));
+      // skip
       break;
     }
   }
@@ -267,9 +265,7 @@ int FLTSpanInfo::deserialize_field(FullLinkTraceExtraInfoId extra_id, const int6
       break;
     }
     default: {
-      ret = OB_NOT_SUPPORTED;
-      LOG_USER_ERROR(OB_NOT_SUPPORTED, "this extra info id");
-      OB_LOG(WARN,"invalid extra info id", K(extra_id));
+      // skip
       break;
     }
   }
@@ -301,9 +297,7 @@ int FLTDrvSpan::deserialize_field(FullLinkTraceExtraInfoId extra_id, const int64
       break;
     }
     default: {
-      ret = OB_NOT_SUPPORTED;
-      LOG_USER_ERROR(OB_NOT_SUPPORTED, "this extra info id");
-      OB_LOG(WARN,"invalid extra info id", K(extra_id));
+      // skip
       break;
     }
   }
@@ -362,9 +356,7 @@ int FLTAppInfo::deserialize_field(FullLinkTraceExtraInfoId extra_id, const int64
       break;
     }
     default: {
-      ret = OB_NOT_SUPPORTED;
-      LOG_USER_ERROR(OB_NOT_SUPPORTED, "this extra info id");
-      OB_LOG(WARN,"invalid extra info id", K(extra_id));
+      // skip
       break;
     }
   }
@@ -434,9 +426,7 @@ int FLTQueryInfo::deserialize_field(FullLinkTraceExtraInfoId extra_id, const int
      break;
     }
     default: {
-      ret = OB_NOT_SUPPORTED;
-      LOG_USER_ERROR(OB_NOT_SUPPORTED, "this extra info id");
-      OB_LOG(WARN,"invalid extra info id", K(extra_id));
+      // skip
       break;
     }
   }
@@ -472,8 +462,7 @@ int FLTShowTrace::deserialize_field(FullLinkTraceExtraInfoId extra_id, const int
       break;
     }
     default: {
-      ret = OB_NOT_SUPPORTED;
-      OB_LOG(WARN,"invalid extra info id", K(extra_id));
+      // skip
       break;
     }
   }

diff --git a/src/sql/monitor/flt/ob_flt_utils.cpp b/src/sql/monitor/flt/ob_flt_utils.cpp
@@ -307,14 +307,12 @@ namespace sql
           }
           case FLT_TYPE_QUERY_INFO: {
             // this extra info is written by server, and driver/proxy cannot send this to server;
-            ret = OB_NOT_SUPPORTED;
-            LOG_WARN("invalid extra_type", K(extra_type), K(ret));
+            // skip
             break;
           }
           case FLT_TYPE_CONTROL_INFO: {
             // this extra info is written by server, and driver/proxy cannot send this to server;
-            ret = OB_NOT_SUPPORTED;
-            LOG_WARN("invalid extra_type", K(extra_type), K(ret));
+            // skip
             break;
           }
           case FLT_TYPE_SPAN_INFO: {
@@ -347,8 +345,7 @@ namespace sql
             break;
           }
           default: {
-            ret = OB_NOT_SUPPORTED;
-            LOG_WARN("invalid extra_type", K(extra_type), K(ret));
+            // skip
             break;
           }
         } // switch ends