letsencrypt
/
boulder
mirror of https://github.com/letsencrypt/boulder.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

ocsp-updater: Set database Tx Isolation Level (#5634) 

Set ocsp-updater's database transaction isolation level to
`READ-UNCOMMITTED`. We don't need consistent reads,
as the only thing which should be updating these rows is
ocsp-updater itself.
This commit is contained in:
Samantha 2021-09-07 11:42:12 -07:00 committed by GitHub
parent 1a4b40d2ce
commit 9874d2018d
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 12 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
cmd/ocsp-updater/main.go
View File

@ -10,6 +10,7 @@ import (
"strings"
"time"
"github.com/go-sql-driver/mysql"
"github.com/honeycombio/beeline-go"
"github.com/jmhodges/clock"
"github.com/prometheus/client_golang/prometheus"
@ -404,8 +405,18 @@ func main() {
dbDSN, err := databaseConfig.URL()
cmd.FailOnError(err, "Couldn't load DB URL")
conf, err := mysql.ParseDSN(dbDSN)
cmd.FailOnError(err, "Couldn't parse DB URL as DSN")
// Set transaction isolation level to READ UNCOMMITTED. This trades
// consistency for performance.
if len(conf.Params) == 0 {
conf.Params = make(map[string]string)
}
conf.Params["tx_isolation"] = "'READ-UNCOMMITTED'"
dbDSN = conf.FormatDSN()
dbMap, err := sa.NewDbMap(dbDSN, dbSettings)
cmd.FailOnError(err, "Couldn't connect to SA database")
cmd.FailOnError(err, "Could not connect to database")
dbAddr, dbUser, err := databaseConfig.DSNAddressAndUser()
cmd.FailOnError(err, "Could not determine address or user of DB DSN")