Kiu/VictoriaMetrics
Kiu/VictoriaMetrics
1
0
Fork 0
mirror of https://github.com/VictoriaMetrics/VictoriaMetrics.git synced 2024-11-23 12:31:07 +01:00
Code Issues Actions 8 Packages Projects Releases Wiki Activity

lib/storage: rename *AssistedMerges to *AssistedMergesCount in order to make these field names less misleading

Browse Source 
These fields are counters, not gauges, so adding Count suffix to them makes easier to understand this while reading the code
This commit is contained in:
Aliaksandr Valialkin 2024-01-25 10:18:37 +02:00
parent 806c07ddd5
commit c3a585cfe5
No known key found for this signature in database
GPG Key ID: 52C003EE2BCDB9EB
2 changed files with 10 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
app/vmstorage/main.go
View File

@ -496,8 +496,8 @@ func writeStorageMetrics(w io.Writer, strg *storage.Storage) {
metrics.WriteCounterUint64(w, `vm_composite_filter_success_conversions_total`, idbm.CompositeFilterSuccessConversions)
metrics.WriteCounterUint64(w, `vm_composite_filter_missing_conversions_total`, idbm.CompositeFilterMissingConversions)
metrics.WriteCounterUint64(w, `vm_assisted_merges_total{type="storage/inmemory"}`, tm.InmemoryAssistedMerges)
metrics.WriteCounterUint64(w, `vm_assisted_merges_total{type="storage/small"}`, tm.SmallAssistedMerges)
metrics.WriteCounterUint64(w, `vm_assisted_merges_total{type="storage/inmemory"}`, tm.InmemoryAssistedMergesCount)
metrics.WriteCounterUint64(w, `vm_assisted_merges_total{type="storage/small"}`, tm.SmallAssistedMergesCount)
metrics.WriteCounterUint64(w, `vm_assisted_merges_total{type="indexdb/inmemory"}`, idbm.InmemoryAssistedMergesCount)
metrics.WriteCounterUint64(w, `vm_assisted_merges_total{type="indexdb/file"}`, idbm.FileAssistedMergesCount)

16
lib/storage/partition.go
View File

@ -117,8 +117,8 @@ type partition struct {
smallRowsDeleted uint64
bigRowsDeleted uint64
inmemoryAssistedMerges uint64
smallAssistedMerges uint64
inmemoryAssistedMergesCount uint64
smallAssistedMergesCount uint64
mergeIdx uint64
@ -350,8 +350,8 @@ type partitionMetrics struct {
SmallPartsRefCount uint64
BigPartsRefCount uint64
InmemoryAssistedMerges uint64
SmallAssistedMerges uint64
InmemoryAssistedMergesCount uint64
SmallAssistedMergesCount uint64
}
// TotalRowsCount returns total number of rows in tm.
@ -415,8 +415,8 @@ func (pt *partition) UpdateMetrics(m *partitionMetrics) {
m.SmallRowsDeleted += atomic.LoadUint64(&pt.smallRowsDeleted)
m.BigRowsDeleted += atomic.LoadUint64(&pt.bigRowsDeleted)
m.InmemoryAssistedMerges += atomic.LoadUint64(&pt.inmemoryAssistedMerges)
m.SmallAssistedMerges += atomic.LoadUint64(&pt.smallAssistedMerges)
m.InmemoryAssistedMergesCount += atomic.LoadUint64(&pt.inmemoryAssistedMergesCount)
m.SmallAssistedMergesCount += atomic.LoadUint64(&pt.smallAssistedMergesCount)
}
// AddRows adds the given rows to the partition pt.
@ -641,7 +641,7 @@ func (pt *partition) assistedMergeForInmemoryParts() {
return
}
atomic.AddUint64(&pt.inmemoryAssistedMerges, 1)
atomic.AddUint64(&pt.inmemoryAssistedMergesCount, 1)
err := pt.mergeInmemoryParts()
if err == nil {
return
@ -660,7 +660,7 @@ func (pt *partition) assistedMergeForSmallParts() {
return
}
atomic.AddUint64(&pt.smallAssistedMerges, 1)
atomic.AddUint64(&pt.smallAssistedMergesCount, 1)
err := pt.mergeExistingParts(false)
if err == nil {
return