Skip to content

Commit

Permalink
fix: update default connection to 100 (flyteorg#135)
Browse files Browse the repository at this point in the history
  • Loading branch information
yindia authored May 12, 2022
1 parent 2a94c0c commit 70688ae
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion database/config.go
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ const (

var defaultConfig = &DbConfig{
MaxIdleConnections: 10,
MaxOpenConnections: 1000,
MaxOpenConnections: 100,
ConnMaxLifeTime: config.Duration{Duration: time.Hour},
Postgres: PostgresConfig{
Port: 5432,
Expand Down
2 changes: 1 addition & 1 deletion database/config_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ func TestParseDatabaseConfig(t *testing.T) {
assert.NoError(t, accessor.UpdateConfig(context.Background()))

assert.Equal(t, false, GetConfig().EnableForeignKeyConstraintWhenMigrating)
assert.Equal(t, 1000, GetConfig().MaxOpenConnections)
assert.Equal(t, 100, GetConfig().MaxOpenConnections)
assert.Equal(t, 10, GetConfig().MaxIdleConnections)
assert.Equal(t, config.Duration{Duration: 3600000000000}, GetConfig().ConnMaxLifeTime)

Expand Down

0 comments on commit 70688ae

Please sign in to comment.