Fix IllegalArgumentException: Incorrect number of labels. (#13429)

Author: wu-sheng

docs/en/changes/changes.md

@@ -63,6 +63,8 @@
 * Bump up netty to 4.2.4.Final.
 * Bump up commons-lang to 3.18.0.
 * BanyanDB: support group `replicas` and `user/password` for basic authentication.
+* BanyanDB: fix Zipkin query missing tag `QUERY`.
+* Fix `IllegalArgumentException: Incorrect number of labels`, tags in the `LogReportServiceHTTPHandler` and `LogReportServiceGrpcHandler` inconsistent with `LogHandler`.
 * BanyanDB: fix Zipkin query by `annotationQuery`
 
 #### UI

oap-server/server-receiver-plugin/skywalking-log-receiver-plugin/src/main/java/org/apache/skywalking/oap/server/receiver/log/provider/handler/grpc/LogReportServiceGrpcHandler.java

@@ -52,11 +52,16 @@ public LogReportServiceGrpcHandler(final ModuleManager moduleManager) {
                                                .getService(ILogAnalyzerService.class);
 
         histogram = metricsCreator.createHistogramMetric(
-            "log_in_latency", "The process latency of log",
-            new MetricsTag.Keys("protocol"), new MetricsTag.Values("grpc")
+            "log_in_latency",
+            "The process latency of log",
+            new MetricsTag.Keys("protocol", "data_format"),
+            new MetricsTag.Values("grpc", "protobuf")
         );
-        errorCounter = metricsCreator.createCounter("log_analysis_error_count", "The error number of log analysis",
-                                                    new MetricsTag.Keys("protocol"), new MetricsTag.Values("grpc")
+        errorCounter = metricsCreator.createCounter(
+            "log_analysis_error_count",
+            "The error number of log analysis",
+            new MetricsTag.Keys("protocol", "data_format"),
+            new MetricsTag.Values("grpc", "protobuf")
         );
     }
 

oap-server/server-receiver-plugin/skywalking-log-receiver-plugin/src/main/java/org/apache/skywalking/oap/server/receiver/log/provider/handler/rest/LogReportServiceHTTPHandler.java

@@ -50,11 +50,13 @@ public LogReportServiceHTTPHandler(final ModuleManager moduleManager) {
 
         histogram = metricsCreator.createHistogramMetric(
             "log_in_latency", "The process latency of log",
-            new MetricsTag.Keys("protocol"), new MetricsTag.Values("http")
+            new MetricsTag.Keys("protocol", "data_format"),
+            new MetricsTag.Values("http", "json")
         );
         errorCounter = metricsCreator.createCounter(
             "log_analysis_error_count", "The error number of log analysis",
-            new MetricsTag.Keys("protocol"), new MetricsTag.Values("http")
+            new MetricsTag.Keys("protocol", "data_format"),
+            new MetricsTag.Values("http", "json")
         );
     }