refactor(repository): change SaveSpans method signature to accept a s…

…lice instead of a pointer to a slice for better usability

refactor(trace_service): update ProcessSpan method to accept a slice instead of a pointer to a slice for consistency
refactor(trace_server): modify Export method to pass spans as a slice instead of a pointer to a slice for improved clarity

internal/repository/interfaces.go

@@ -7,5 +7,5 @@ import (
 
 type Repository interface {
 	SaveTraces(ctx context.Context, traces []*models.Trace) error
-	SaveSpans(ctx context.Context, spans *[]models.Span) error
+	SaveSpans(ctx context.Context, spans []models.Span) error
 }

internal/repository/mongodb.go

@@ -62,13 +62,13 @@ func (r *MongoRepository) SaveTraces(ctx context.Context, traces []*models.Trace
 	return nil
 }
 
-func (r *MongoRepository) SaveSpans(ctx context.Context, spans *[]models.Span) error {
-	if len(*spans) == 0 {
+func (r *MongoRepository) SaveSpans(ctx context.Context, spans []models.Span) error {
+	if len(spans) == 0 {
 		return nil
 	}
 
-	documents := make([]interface{}, len(*spans))
-	for i, span := range *spans {
+	documents := make([]interface{}, len(spans))
+	for i, span := range spans {
 		doc := bson.M{
 			"id":                span.ID,
 			"traceId":           span.TraceID,

internal/server/trace_server.go

@@ -26,7 +26,7 @@ func (s *TraceServer) Export(ctx context.Context, req *collectorpb.ExportTraceSe
 	for _, resourceSpans := range req.ResourceSpans {
 		spans := telemetry.ConvertResourceSpansToSpans(resourceSpans)
 
-		err := s.traceService.ProcessSpan(ctx, &spans)
+		err := s.traceService.ProcessSpan(ctx, spans)
 		if err != nil {
 			fmt.Printf("Failed to process span: %v\n", err)
 			return nil, status.Errorf(codes.Internal, "Failed to process span: %v", err)

internal/service/trace_service.go

@@ -18,6 +18,6 @@ func (s *TraceService) ProcessTrace(ctx context.Context, traces []*models.Trace)
 	return s.repo.SaveTraces(ctx, traces)
 }
 
-func (s *TraceService) ProcessSpan(ctx context.Context, spans *[]models.Span) error {
+func (s *TraceService) ProcessSpan(ctx context.Context, spans []models.Span) error {
 	return s.repo.SaveSpans(ctx, spans)
 }