fix review issues

spans/discovery.go

@@ -1,6 +1,7 @@
 package spans
 
 import (
+	"fmt"
 	"github.com/ydb-platform/ydb-go-sdk/v3/internal/kv"
 	"github.com/ydb-platform/ydb-go-sdk/v3/trace"
 )
@@ -17,15 +18,16 @@ func discovery(adapter Adapter) (t trace.Discovery) {
 			)
 
 			return func(info trace.DiscoveryDiscoverDoneInfo) {
-				endpoints := make([]string, len(info.Endpoints))
-				for i, e := range info.Endpoints {
-					endpoints[i] = e.String()
+				if info.Error != nil {
+					start.Error(info.Error)
+				} else {
+					endpoints := make([]string, len(info.Endpoints))
+					for i, e := range info.Endpoints {
+						endpoints[i] = e.String()
+					}
+					start.Log(fmt.Sprintf("endpoints=%v", endpoints))
 				}
-				finish(
-					start,
-					info.Error,
-					kv.Strings("endpoints", endpoints),
-				)
+				start.End()
 			}
 		}
 

spans/driver.go

@@ -298,11 +298,11 @@ func driver(adapter Adapter) trace.Driver { //nolint:gocyclo,funlen
 				for i, e := range info.Dropped {
 					dropped[i] = e.String()
 				}
+				start.Log(fmt.Sprintf("endpoints=%v", endpoints))
+				start.Log(fmt.Sprintf("added=%v", added))
+				start.Log(fmt.Sprintf("dropped=%v", dropped))
 				start.End(
 					kv.String("local_dc", info.LocalDC),
-					kv.Strings("endpoints", endpoints),
-					kv.Strings("added", added),
-					kv.Strings("dropped", dropped),
 				)
 			}
 		},

spans/scripting.go

@@ -14,7 +14,6 @@ func scripting(adapter Adapter) (t trace.Scripting) {
 				info.Context,
 				info.Call.FunctionID(),
 				kv.String("query", info.Query),
-				kv.String("params", safeStringer(info.Parameters)),
 			)
 
 			return func(info trace.ScriptingExecuteDoneInfo) {
@@ -47,7 +46,6 @@ func scripting(adapter Adapter) (t trace.Scripting) {
 				info.Context,
 				info.Call.FunctionID(),
 				kv.String("query", info.Query),
-				kv.String("params", safeStringer(info.Parameters)),
 			)
 
 			return func(