mudhorn/prometheus
1
0
Fork 0
mirror of https://github.com/prometheus/prometheus.git synced 2025-03-05 20:59:13 -08:00
Code Issues Actions 4 Packages Projects Releases Wiki Activity

Merge pull request #21 from grafana/reuse-expand-postings-method-in-new-postings-cloner

Browse source 
Use `ExpandPostings()` in `NewPostingsCloner()`
This commit is contained in:
Marco Pracucci 2021-10-13 17:01:24 +02:00 committed by GitHub
parent 8bc1b41795 14cb8d8696
commit fbd1b4fb84
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
tsdb/index/postings.go
View file

@ -807,11 +807,8 @@ type PostingsCloner struct {
// The instance provided shouldn't have been used before (no Next() calls should have been done)
// and it shouldn't be used once provided to the PostingsCloner.
func NewPostingsCloner(p Postings) *PostingsCloner {
var ids []uint64
for p.Next() {
ids = append(ids, p.At())
}
return &PostingsCloner{ids: ids, err: p.Err()}
ids, err := ExpandPostings(p)
return &PostingsCloner{ids: ids, err: err}
}
// Clone returns another independent Postings instance.