fix

wankai123 · wankai123 · commit 4353db1fbe7b · 2026-03-02T14:27:29.000+08:00
diff --git a/oap-server/server-query-plugin/traceql-plugin/src/main/java/org/apache/skywalking/oap/query/traceql/handler/ZipkinTraceQLApiHandler.java b/oap-server/server-query-plugin/traceql-plugin/src/main/java/org/apache/skywalking/oap/query/traceql/handler/ZipkinTraceQLApiHandler.java
@@ -326,7 +326,7 @@ private void parseTagsParameter(Optional<String> tags, QueryRequest.Builder quer
      */
     private Long parseDurationToMicros(String durationStr) {
         if (durationStr == null || durationStr.isEmpty()) {
-            return null;
+            throw new IllegalArgumentException("Duration string cannot be null or empty");
         }
 
         try {
diff --git a/test/e2e-v2/cases/traceql/zipkin/expected/tag-values-service.yml b/test/e2e-v2/cases/traceql/zipkin/expected/tag-values-service.yml
@@ -14,7 +14,9 @@
 # limitations under the License.
 
 tagValues:
+  {{- contains .tagValues }}
   - type: string
     value: backend
   - type: string
     value: frontend
+  {{- end }}

Original file line number	Diff line number	Diff line change
`@@ -326,7 +326,7 @@ private void parseTagsParameter(Optional<String> tags, QueryRequest.Builder quer`
`326`	`326`	`*/`
`327`	`327`	`private Long parseDurationToMicros(String durationStr) {`
`328`	`328`	`if (durationStr == null \|\| durationStr.isEmpty()) {`
`329`		`- return null;`
	`329`	`+ throw new IllegalArgumentException("Duration string cannot be null or empty");`
`330`	`330`	`}`
`331`	`331`
`332`	`332`	`try {`