Skip to content

Commit 0c9c840

Browse files
zhouselenamatthewdale
zhouselena
and
matthewdale
authored
GODRIVER-3102: Perf comparison (#2134)
Co-authored-by: Matt Dale <[email protected]>
1 parent d3c5ace commit 0c9c840

File tree

7 files changed

+632
-0
lines changed

7 files changed

+632
-0
lines changed

‎.evergreen/config.yml

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -285,6 +285,16 @@ functions:
285285
echo "Response Body: $response_body"
286286
echo "HTTP Status: $http_status"
287287
288+
send-perf-pr-comment:
289+
- command: subprocess.exec
290+
type: test
291+
params:
292+
binary: bash
293+
env:
294+
VERSION_ID: ${version_id}
295+
include_expansions_in_env: [PERF_URI_PRIVATE_ENDPOINT]
296+
args: [*task-runner, perf-pr-comment]
297+
288298
run-enterprise-auth-tests:
289299
- command: ec2.assume_role
290300
params:
@@ -684,6 +694,7 @@ tasks:
684694
binary: bash
685695
args: [*task-runner, driver-benchmark]
686696
- func: send-perf-data
697+
- func: send-perf-pr-comment
687698

688699
- name: test-standalone-noauth-nossl
689700
tags: ["test", "standalone"]

‎Taskfile.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -70,6 +70,8 @@ tasks:
7070

7171
pr-task: bash etc/pr-task.sh
7272

73+
perf-pr-comment: bash etc/perf-pr-comment.sh
74+
7375
# Lint with various GOOS and GOARCH tasks to catch static analysis failures that may only affect
7476
# specific operating systems or architectures. For example, staticcheck will only check for 64-bit
7577
# alignment of atomically accessed variables on 32-bit architectures (see

‎etc/perf-pr-comment.sh

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,9 @@
1+
#!/usr/bin/env bash
2+
# perf-pr-comment
3+
# Generates a report of Go Driver perf changes for the current branch.
4+
5+
set -eux
6+
7+
pushd ./internal/cmd/perfcomp >/dev/null || exist
8+
GOWORK=off go run main.go --project="mongo-go-driver" ${VERSION_ID}
9+
popd >/dev/null

‎internal/cmd/perfcomp/energystatistics_test.go

Lines changed: 98 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,98 @@
1+
// Copyright (C) MongoDB, Inc. 2025-present.
2+
//
3+
// Licensed under the Apache License, Version 2.0 (the "License"); you may
4+
// not use this file except in compliance with the License. You may obtain
5+
// a copy of the License at http://www.apache.org/licenses/LICENSE-2.0
6+
7+
package main
8+
9+
import (
10+
"testing"
11+
12+
"github.com/stretchr/testify/assert"
13+
"gonum.org/v1/gonum/mat"
14+
)
15+
16+
func createTestVectors(start1 int, stop1 int, step1 int, start2 int, stop2 int, step2 int) (*mat.Dense, *mat.Dense) {
17+
xData := []float64{}
18+
yData := []float64{}
19+
20+
for i := start1; i < stop1; i += step1 {
21+
xData = append(xData, float64(i))
22+
}
23+
for j := start2; j < stop2; j += step2 {
24+
yData = append(yData, float64(j))
25+
}
26+
27+
x := mat.NewDense(len(xData), 1, xData)
28+
y := mat.NewDense(len(yData), 1, yData)
29+
30+
return x, y
31+
}
32+
33+
func TestEnergyStatistics(t *testing.T) {
34+
t.Run("similar distributions should have small e,t,h values ", func(t *testing.T) {
35+
x, y := createTestVectors(1, 100, 1, 1, 105, 1)
36+
e, tstat, h, _ := getEnergyStatistics(x, y)
37+
38+
del := 1e-3
39+
// Limit precision of comparison to 3 digits after the decimal.
40+
assert.InDelta(t, 0.160, e, del) // |0.160 - e| < 0.001
41+
assert.InDelta(t, 8.136, tstat, del)
42+
assert.InDelta(t, 0.002, h, del)
43+
})
44+
45+
t.Run("different distributions should have large e,t,h values", func(t *testing.T) {
46+
x, y := createTestVectors(1, 100, 1, 10000, 13000, 14)
47+
e, tstat, h, _ := getEnergyStatistics(x, y)
48+
del := 1e-3
49+
50+
assert.InDelta(t, 21859.691, e, del)
51+
assert.InDelta(t, 1481794.709, tstat, del)
52+
assert.InDelta(t, 0.954, h, del)
53+
})
54+
55+
t.Run("uni-variate distributions", func(t *testing.T) {
56+
x, y := createTestVectors(1, 300, 1, 1000, 5000, 10)
57+
e, tstat, h, _ := getEnergyStatistics(x, y)
58+
del := 1e-3
59+
60+
assert.InDelta(t, 4257.009, e, del)
61+
assert.InDelta(t, 728381.015, tstat, del)
62+
assert.InDelta(t, 0.748, h, del)
63+
})
64+
65+
t.Run("equal distributions should have all 0 values", func(t *testing.T) {
66+
x := mat.NewDense(10, 1, []float64{1, 1, 1, 1, 1, 1, 1, 1, 1, 1})
67+
y := mat.NewDense(1, 1, []float64{1})
68+
69+
e, tstat, h, _ := getEnergyStatistics(x, y)
70+
71+
assert.Equal(t, 0.0, e)
72+
assert.Equal(t, 0.0, tstat)
73+
assert.Equal(t, 0.0, h)
74+
})
75+
76+
t.Run("energy stats returns errors on malformed input", func(t *testing.T) {
77+
x := mat.NewDense(2, 2, make([]float64, 4))
78+
y := mat.NewDense(2, 3, make([]float64, 6))
79+
80+
_, _, _, err := getEnergyStatistics(x, y)
81+
assert.NotEqual(t, nil, err)
82+
assert.ErrorContains(t, err, "both inputs must have the same number of columns")
83+
84+
x.Reset()
85+
y = &mat.Dense{}
86+
87+
_, _, _, err = getEnergyStatistics(x, y)
88+
assert.NotEqual(t, nil, err)
89+
assert.ErrorContains(t, err, "inputs cannot be empty")
90+
91+
x = mat.NewDense(2, 2, make([]float64, 4))
92+
y = mat.NewDense(3, 2, make([]float64, 6))
93+
94+
_, _, _, err = getEnergyStatistics(x, y)
95+
assert.NotEqual(t, nil, err)
96+
assert.ErrorContains(t, err, "both inputs must be column vectors")
97+
})
98+
}

‎internal/cmd/perfcomp/go.mod

Lines changed: 28 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,28 @@
1+
module go.mongodb.go/mongo-driver/v2/internal/cmd/perfcomp
2+
3+
go 1.23.0
4+
5+
toolchain go1.23.10
6+
7+
replace go.mongodb.org/mongo-driver/v2 => ../../../
8+
9+
require (
10+
github.com/stretchr/testify v1.10.0
11+
go.mongodb.org/mongo-driver/v2 v2.2.2
12+
gonum.org/v1/gonum v0.16.0
13+
)
14+
15+
require (
16+
github.com/davecgh/go-spew v1.1.1 // indirect
17+
github.com/golang/snappy v1.0.0 // indirect
18+
github.com/klauspost/compress v1.16.7 // indirect
19+
github.com/pmezard/go-difflib v1.0.0 // indirect
20+
github.com/xdg-go/pbkdf2 v1.0.0 // indirect
21+
github.com/xdg-go/scram v1.1.2 // indirect
22+
github.com/xdg-go/stringprep v1.0.4 // indirect
23+
github.com/youmark/pkcs8 v0.0.0-20240726163527-a2c0da244d78 // indirect
24+
golang.org/x/crypto v0.33.0 // indirect
25+
golang.org/x/sync v0.12.0 // indirect
26+
golang.org/x/text v0.23.0 // indirect
27+
gopkg.in/yaml.v3 v3.0.1 // indirect
28+
)

‎internal/cmd/perfcomp/go.sum

Lines changed: 56 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,56 @@
1+
github.com/davecgh/go-spew v1.1.1 h1:vj9j/u1bqnvCEfJOwUhtlOARqs3+rkHYY13jYWTU97c=
2+
github.com/davecgh/go-spew v1.1.1/go.mod h1:J7Y8YcW2NihsgmVo/mv3lAwl/skON4iLHjSsI+c5H38=
3+
github.com/golang/snappy v1.0.0 h1:Oy607GVXHs7RtbggtPBnr2RmDArIsAefDwvrdWvRhGs=
4+
github.com/golang/snappy v1.0.0/go.mod h1:/XxbfmMg8lxefKM7IXC3fBNl/7bRcc72aCRzEWrmP2Q=
5+
github.com/google/go-cmp v0.6.0 h1:ofyhxvXcZhMsU5ulbFiLKl/XBFqE1GSq7atu8tAmTRI=
6+
github.com/google/go-cmp v0.6.0/go.mod h1:17dUlkBOakJ0+DkrSSNjCkIjxS6bF9zb3elmeNGIjoY=
7+
github.com/klauspost/compress v1.16.7 h1:2mk3MPGNzKyxErAw8YaohYh69+pa4sIQSC0fPGCFR9I=
8+
github.com/klauspost/compress v1.16.7/go.mod h1:ntbaceVETuRiXiv4DpjP66DpAtAGkEQskQzEyD//IeE=
9+
github.com/pmezard/go-difflib v1.0.0 h1:4DBwDE0NGyQoBHbLQYPwSUPoCMWR5BEzIk/f1lZbAQM=
10+
github.com/pmezard/go-difflib v1.0.0/go.mod h1:iKH77koFhYxTK1pcRnkKkqfTogsbg7gZNVY4sRDYZ/4=
11+
github.com/stretchr/testify v1.10.0 h1:Xv5erBjTwe/5IxqUQTdXv5kgmIvbHo3QQyRwhJsOfJA=
12+
github.com/stretchr/testify v1.10.0/go.mod h1:r2ic/lqez/lEtzL7wO/rwa5dbSLXVDPFyf8C91i36aY=
13+
github.com/xdg-go/pbkdf2 v1.0.0 h1:Su7DPu48wXMwC3bs7MCNG+z4FhcyEuz5dlvchbq0B0c=
14+
github.com/xdg-go/pbkdf2 v1.0.0/go.mod h1:jrpuAogTd400dnrH08LKmI/xc1MbPOebTwRqcT5RDeI=
15+
github.com/xdg-go/scram v1.1.2 h1:FHX5I5B4i4hKRVRBCFRxq1iQRej7WO3hhBuJf+UUySY=
16+
github.com/xdg-go/scram v1.1.2/go.mod h1:RT/sEzTbU5y00aCK8UOx6R7YryM0iF1N2MOmC3kKLN4=
17+
github.com/xdg-go/stringprep v1.0.4 h1:XLI/Ng3O1Atzq0oBs3TWm+5ZVgkq2aqdlvP9JtoZ6c8=
18+
github.com/xdg-go/stringprep v1.0.4/go.mod h1:mPGuuIYwz7CmR2bT9j4GbQqutWS1zV24gijq1dTyGkM=
19+
github.com/youmark/pkcs8 v0.0.0-20240726163527-a2c0da244d78 h1:ilQV1hzziu+LLM3zUTJ0trRztfwgjqKnBWNtSRkbmwM=
20+
github.com/youmark/pkcs8 v0.0.0-20240726163527-a2c0da244d78/go.mod h1:aL8wCCfTfSfmXjznFBSZNN13rSJjlIOI1fUNAtF7rmI=
21+
github.com/yuin/goldmark v1.4.13/go.mod h1:6yULJ656Px+3vBD8DxQVa3kxgyrAnzto9xy5taEt/CY=
22+
golang.org/x/crypto v0.0.0-20190308221718-c2843e01d9a2/go.mod h1:djNgcEr1/C05ACkg1iLfiJU5Ep61QUkGW8qpdssI0+w=
23+
golang.org/x/crypto v0.0.0-20210921155107-089bfa567519/go.mod h1:GvvjBRRGRdwPK5ydBHafDWAxML/pGHZbMvKqRZ5+Abc=
24+
golang.org/x/crypto v0.33.0 h1:IOBPskki6Lysi0lo9qQvbxiQ+FvsCC/YWOecCHAixus=
25+
golang.org/x/crypto v0.33.0/go.mod h1:bVdXmD7IV/4GdElGPozy6U7lWdRXA4qyRVGJV57uQ5M=
26+
golang.org/x/mod v0.6.0-dev.0.20220419223038-86c51ed26bb4/go.mod h1:jJ57K6gSWd91VN4djpZkiMVwK6gcyfeH4XE8wZrZaV4=
27+
golang.org/x/net v0.0.0-20190620200207-3b0461eec859/go.mod h1:z5CRVTTTmAJ677TzLLGU+0bjPO0LkuOLi4/5GtJWs/s=
28+
golang.org/x/net v0.0.0-20210226172049-e18ecbb05110/go.mod h1:m0MpNAwzfU5UDzcl9v0D8zg8gWTRqZa9RBIspLL5mdg=
29+
golang.org/x/net v0.0.0-20220722155237-a158d28d115b/go.mod h1:XRhObCWvk6IyKnWLug+ECip1KBveYUHfp+8e9klMJ9c=
30+
golang.org/x/sync v0.0.0-20190423024810-112230192c58/go.mod h1:RxMgew5VJxzue5/jJTE5uejpjVlOe/izrB70Jof72aM=
31+
golang.org/x/sync v0.0.0-20220722155255-886fb9371eb4/go.mod h1:RxMgew5VJxzue5/jJTE5uejpjVlOe/izrB70Jof72aM=
32+
golang.org/x/sync v0.12.0 h1:MHc5BpPuC30uJk597Ri8TV3CNZcTLu6B6z4lJy+g6Jw=
33+
golang.org/x/sync v0.12.0/go.mod h1:1dzgHSNfp02xaA81J2MS99Qcpr2w7fw1gpm99rleRqA=
34+
golang.org/x/sys v0.0.0-20190215142949-d0b11bdaac8a/go.mod h1:STP8DvDyc/dI5b8T5hshtkjS+E42TnysNCUPdjciGhY=
35+
golang.org/x/sys v0.0.0-20201119102817-f84b799fce68/go.mod h1:h1NjWce9XRLGQEsW7wpKNCjG9DtNlClVuFLEZdDNbEs=
36+
golang.org/x/sys v0.0.0-20210615035016-665e8c7367d1/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
37+
golang.org/x/sys v0.0.0-20220520151302-bc2c85ada10a/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
38+
golang.org/x/sys v0.0.0-20220722155257-8c9f86f7a55f/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
39+
golang.org/x/term v0.0.0-20201126162022-7de9c90e9dd1/go.mod h1:bj7SfCRtBDWHUb9snDiAeCFNEtKQo2Wmx5Cou7ajbmo=
40+
golang.org/x/term v0.0.0-20210927222741-03fcf44c2211/go.mod h1:jbD1KX2456YbFQfuXm/mYQcufACuNUgVhRMnK/tPxf8=
41+
golang.org/x/text v0.3.0/go.mod h1:NqM8EUOU14njkJ3fqMW+pc6Ldnwhi/IjpwHt7yyuwOQ=
42+
golang.org/x/text v0.3.3/go.mod h1:5Zoc/QRtKVWzQhOtBMvqHzDpF6irO9z98xDceosuGiQ=
43+
golang.org/x/text v0.3.7/go.mod h1:u+2+/6zg+i71rQMx5EYifcz6MCKuco9NR6JIITiCfzQ=
44+
golang.org/x/text v0.3.8/go.mod h1:E6s5w1FMmriuDzIBO73fBruAKo1PCIq6d2Q6DHfQ8WQ=
45+
golang.org/x/text v0.23.0 h1:D71I7dUrlY+VX0gQShAThNGHFxZ13dGLBHQLVl1mJlY=
46+
golang.org/x/text v0.23.0/go.mod h1:/BLNzu4aZCJ1+kcD0DNRotWKage4q2rGVAg4o22unh4=
47+
golang.org/x/tools v0.0.0-20180917221912-90fa682c2a6e/go.mod h1:n7NCudcB/nEzxVGmLbDWY5pfWTLqBcC2KZ6jyYvM4mQ=
48+
golang.org/x/tools v0.0.0-20191119224855-298f0cb1881e/go.mod h1:b+2E5dAYhXwXZwtnZ6UAqBI28+e2cm9otk0dWdXHAEo=
49+
golang.org/x/tools v0.1.12/go.mod h1:hNGJHUnrk76NpqgfD5Aqm5Crs+Hm0VOH/i9J2+nxYbc=
50+
golang.org/x/xerrors v0.0.0-20190717185122-a985d3407aa7/go.mod h1:I/5z698sn9Ka8TeJc9MKroUUfqBBauWjQqLJ2OPfmY0=
51+
gonum.org/v1/gonum v0.16.0 h1:5+ul4Swaf3ESvrOnidPp4GZbzf0mxVQpDCYUQE7OJfk=
52+
gonum.org/v1/gonum v0.16.0/go.mod h1:fef3am4MQ93R2HHpKnLk4/Tbh/s0+wqD5nfa6Pnwy4E=
53+
gopkg.in/check.v1 v0.0.0-20161208181325-20d25e280405 h1:yhCVgyC4o1eVCa2tZl7eS0r+SDo693bJlVdllGtEeKM=
54+
gopkg.in/check.v1 v0.0.0-20161208181325-20d25e280405/go.mod h1:Co6ibVJAznAaIkqp8huTwlJQCZ016jof/cbN4VW5Yz0=
55+
gopkg.in/yaml.v3 v3.0.1 h1:fxVm/GzAzEWqLHuvctI91KS9hhNmmWOoWu0XTYJS7CA=
56+
gopkg.in/yaml.v3 v3.0.1/go.mod h1:K4uyk7z7BCEPqu6E+C64Yfv1cQ7kz7rIZviUmN+EgEM=

0 commit comments

Comments
 (0)