github-mirrors/VictoriaMetrics
1
0
Fork 0
mirror of https://github.com/VictoriaMetrics/VictoriaMetrics.git synced 2024-11-21 14:44:00 +00:00
Code Issues Projects Releases Packages Wiki Activity

lib/storage: get parts to merge after applying the limit on the number of concurrent merges

Browse source 
This should reduce write amplification under high ingestion rate.
This commit is contained in:
Aliaksandr Valialkin 2019-10-30 00:07:31 +02:00
parent 5d2276dbf7
commit a27e034a40
1 changed files with 10 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
lib/storage/partition.go
View file

@ -868,6 +868,11 @@ type freeSpaceEntry struct {
}
func (pt *partition) mergeBigParts(isFinal bool) error {
bigMergeConcurrencyLimitCh <- struct{}{}
defer func() {
<-bigMergeConcurrencyLimitCh
}()
maxRows := maxRowsByPath(pt.bigPartsPath)
pt.partsLock.Lock()
@ -880,15 +885,18 @@ func (pt *partition) mergeBigParts(isFinal bool) error {
atomic.AddUint64(&pt.bigMergesCount, 1)
atomic.AddUint64(&pt.activeBigMerges, 1)
bigMergeConcurrencyLimitCh <- struct{}{}
err := pt.mergeParts(pws, pt.stopCh)
<-bigMergeConcurrencyLimitCh
atomic.AddUint64(&pt.activeBigMerges, ^uint64(0))
return err
}
func (pt *partition) mergeSmallParts(isFinal bool) error {
smallMergeConcurrencyLimitCh <- struct{}{}
defer func() {
<-smallMergeConcurrencyLimitCh
}()
maxRows := maxRowsByPath(pt.smallPartsPath)
if maxRows > maxRowsPerSmallPart() {
// The output part may go to big part,
@ -909,9 +917,7 @@ func (pt *partition) mergeSmallParts(isFinal bool) error {
atomic.AddUint64(&pt.smallMergesCount, 1)
atomic.AddUint64(&pt.activeSmallMerges, 1)
smallMergeConcurrencyLimitCh <- struct{}{}
err := pt.mergeParts(pws, pt.stopCh)
<-smallMergeConcurrencyLimitCh
atomic.AddUint64(&pt.activeSmallMerges, ^uint64(0))
return err