From e2153f5c164dc865c1188e9ba3887e10d318dcae Mon Sep 17 00:00:00 2001
From: fengwei <fengwei@manus.ai>
Date: Wed, 3 Dec 2025 22:41:44 +0800
Subject: [PATCH] feat(otel): categorize metrics with nil response (#3638)

* Remove metrics with empty data

Because empty data cannot be considered an error.

* redis.Nil uses a separate state.

Better differentiation between values, null values, and errors.
---
 extra/redisotel/metrics.go | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/extra/redisotel/metrics.go b/extra/redisotel/metrics.go
index 77aa5d14..85065402 100644
--- a/extra/redisotel/metrics.go
+++ b/extra/redisotel/metrics.go
@@ -330,6 +330,9 @@ func milliseconds(d time.Duration) float64 {
 
 func statusAttr(err error) attribute.KeyValue {
 	if err != nil {
+		if  err == redis.Nil {
+			return attribute.String("status", "nil")
+		}
 		return attribute.String("status", "error")
 	}
 	return attribute.String("status", "ok")