Skip to content

Commit ec4a3ed

Browse files
committed
fix(telemetry): lower bias in sample bucket calc
Signed-off-by: Babak K. Shandiz <babakks@github.com>
1 parent 571bb1c commit ec4a3ed

1 file changed

Lines changed: 2 additions & 1 deletion

File tree

internal/telemetry/telemetry.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@ package telemetry
33

44
import (
55
"bytes"
6+
"encoding/binary"
67
"encoding/json"
78
"errors"
89
"fmt"
@@ -232,7 +233,7 @@ func NewService(flusher func(SendTelemetryPayload), opts ...telemetryServiceOpti
232233
maps.Copy(commonDimensions, telemetryServiceOpts.additionalDimensions)
233234

234235
hash := uuid.NewSHA1(uuid.Nil, []byte(invocationID))
235-
sampleBucket := hash[0] % 100
236+
sampleBucket := byte(binary.BigEndian.Uint32(hash[:4]) % 100)
236237

237238
s := &service{
238239
flush: flusher,

0 commit comments

Comments
 (0)