cleanups

Author: kevinburkesegment

datadog/client_test.go

@@ -111,7 +111,6 @@ func TestClientWithUseDistributions(t *testing.T) {
 	expectedPacket1 := "request.count:5|c|#answer:42,hello:world\nrequest.dist_rtt:0.1|d|#answer:42,hello:world\n"
 	select {
 	case packet := <-packets:
-		fmt.Println("receive packet", packet)
 		assert.EqualValues(t, expectedPacket1, string(packet))
 	case <-time.After(2 * time.Second):
 		t.Fatal("no response after 2 seconds")
@@ -253,13 +252,10 @@ func startUDPListener(t *testing.T, packets chan []byte) (addr string, closer io
 		t.Fatal(err)
 	}
 
-	fmt.Println("starting UDP goroutine")
 	go func() {
 		for {
 			packetBytes := make([]byte, 1024)
-			fmt.Println("call conn.ReadFrom")
 			n, _, err := conn.ReadFrom(packetBytes)
-			fmt.Println("read", n, err)
 			if n > 0 {
 				packets <- packetBytes[:n]
 			}

datadog/udp.go

@@ -1,7 +1,6 @@
 package datadog
 
 import (
-	"fmt"
 	"net"
 )
 
@@ -19,7 +18,6 @@ func newUDPWriter(addr string) (*udpWriter, error) {
 	if err != nil {
 		return nil, err
 	}
-	fmt.Printf("udp conn: %#v\n", conn)
 	return &udpWriter{conn: conn}, nil
 }
 
@@ -29,7 +27,6 @@ func (w *udpWriter) Write(data []byte) (int, error) {
 }
 
 func (w *udpWriter) Close() error {
-	fmt.Println("call udpWriter.Close")
 	return w.conn.Close()
 }
 

otlp/handler.go

@@ -27,12 +27,11 @@ const (
 	DefaultFlushInterval = 10 * time.Second
 )
 
-// Status: Alpha. This Handler is still in heavy development phase.
+// Status: Alpha. This Handler is still in heavy development phase. Do not use
+// in production.
 //
-//	Do not use in production.
-//
-// Handler implements stats.Handler to be used to forward metrics to an
-// OpenTelemetry destination. Usually an OpenTelemetry Collector.
+// Handler implements stats.Handler to forward metrics to an OpenTelemetry
+// destination. Usually an OpenTelemetry Collector.
 //
 // With the current implementation this Handler is targeting a Prometheus
 // based backend or any backend expecting cumulative values.