Skip to content

Commit

Permalink
storage: fix incorrect listener on concurrent AddSST setting
Browse files Browse the repository at this point in the history
Copy/paste mistake meant that the wrong setting was being used to trigger the re-read of the setting on-update.

Release note (bug fix): fix bug that prevented changes to kv.bulk_io_write.concurrent_addsstable_requests from taking effect.

Release justification: very small, low-risk fix for user-visible bug in existing code.
  • Loading branch information
dt committed Sep 30, 2019
1 parent d714f55 commit 3255968
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion pkg/storage/store.go
Original file line number Diff line number Diff line change
Expand Up @@ -897,7 +897,7 @@ func NewStore(
s.limiters.ConcurrentAddSSTableRequests = limit.MakeConcurrentRequestLimiter(
"addSSTableRequestLimiter", int(addSSTableRequestLimit.Get(&cfg.Settings.SV)),
)
importRequestsLimit.SetOnChange(&cfg.Settings.SV, func() {
addSSTableRequestLimit.SetOnChange(&cfg.Settings.SV, func() {
s.limiters.ConcurrentAddSSTableRequests.SetLimit(int(addSSTableRequestLimit.Get(&cfg.Settings.SV)))
})
s.limiters.ConcurrentRangefeedIters = limit.MakeConcurrentRequestLimiter(
Expand Down

0 comments on commit 3255968

Please sign in to comment.