fix: 100% redis cpu usage

Signed-off-by: Alessandro Yuichi Okimoto <yuichijpn@gmail.com>

Author: cre8ivejp

pkg/redis/v3/redis.go

@@ -338,7 +338,7 @@ func (c *client) Scan(cursor uint64, key string, count int64) (uint64, []string,
 
 	if c.clientType == ClientTypeCluster {
 		// Use cluster-aware pagination
-		keys, newCursor, err = c.scanClusterWithPagination(cursor, key, count)
+		newCursor, keys, err = c.scanClusterWithPagination(cursor, key, count)
 	} else {
 		// Use standard approach for non-cluster client
 		keys, newCursor, err = c.rc.Scan(context.TODO(), cursor, key, count).Result()
@@ -359,16 +359,16 @@ func (c *client) Scan(cursor uint64, key string, count int64) (uint64, []string,
 
 // scanClusterWithPagination implements cursor-based pagination across cluster nodes
 // For cluster clients, we collect keys from all master nodes in a single call to avoid connection issues
-func (c *client) scanClusterWithPagination(cursor uint64, key string, count int64) ([]string, uint64, error) {
+func (c *client) scanClusterWithPagination(cursor uint64, key string, count int64) (uint64, []string, error) {
 	clusterClient, ok := c.rc.(*goredis.ClusterClient)
 	if !ok {
-		return nil, 0, fmt.Errorf("client is not a cluster client")
+		return 0, nil, fmt.Errorf("client is not a cluster client")
 	}
 
 	// For cluster scanning, if cursor is 0, scan all nodes and return all keys
 	// If cursor is non-zero, we've already completed the scan
 	if cursor != 0 {
-		return []string{}, 0, nil
+		return 0, []string{}, nil
 	}
 
 	var allKeys []string
@@ -388,11 +388,11 @@ func (c *client) scanClusterWithPagination(cursor uint64, key string, count int6
 		return nil
 	})
 	if err != nil {
-		return nil, 0, err
+		return 0, nil, err
 	}
 
-	// Return all keys with cursor=1 to indicate completion
-	return allKeys, 1, nil
+	// Return all keys with cursor=0 to indicate completion
+	return 0, allKeys, nil
 }
 
 func (c *client) Get(key string) ([]byte, error) {