Merge pull request #2 from prozz/no-sampling

Sampling doesn't belong here

Author: prozz

emf/logger.go

@@ -4,7 +4,6 @@ import (
 	"encoding/json"
 	"fmt"
 	"io"
-	"math/rand"
 	"os"
 	"strings"
 	"time"
@@ -170,15 +169,6 @@ func (l *Logger) Log() {
 	_, _ = fmt.Fprintln(l.out, string(buf))
 }
 
-// LogSampled samples Log calls based on a rate.
-// Ex. rate of 0.7 means Log will be called 70% of a time.
-func (l *Logger) LogSampled(rate float64) {
-	rand.Seed(time.Now().UnixNano())
-	if rand.Float64() < rate {
-		l.Log()
-	}
-}
-
 // NewContext creates new context for given logger.
 func (l *Logger) NewContext() *Context {
 	c := newContext(l.values)

emf/logger_test.go

@@ -195,25 +195,6 @@ func TestEmf(t *testing.T) {
 	}
 }
 
-func TestLogger_LogSampled(t *testing.T) {
-
-	t.Run("never", func(t *testing.T) {
-		var buf bytes.Buffer
-		emf.NewFor(&buf).Metric("foo", 1).LogSampled(0)
-		if buf.Len() > 0 {
-			t.Fatal("should be empty")
-		}
-	})
-
-	t.Run("always", func(t *testing.T) {
-		var buf bytes.Buffer
-		emf.NewFor(&buf).Metric("foo", 1).LogSampled(100)
-		if buf.Len() == 0 {
-			t.Fatal("should not be empty")
-		}
-	})
-}
-
 func setenv(t *testing.T, env map[string]string) {
 	for k, v := range env {
 		err := os.Setenv(k, v)