2017-04-10 18:59:45 +00:00
|
|
|
// Copyright 2017 The Prometheus Authors
|
|
|
|
// Licensed under the Apache License, Version 2.0 (the "License");
|
|
|
|
// you may not use this file except in compliance with the License.
|
|
|
|
// You may obtain a copy of the License at
|
|
|
|
//
|
|
|
|
// http://www.apache.org/licenses/LICENSE-2.0
|
|
|
|
//
|
|
|
|
// Unless required by applicable law or agreed to in writing, software
|
|
|
|
// distributed under the License is distributed on an "AS IS" BASIS,
|
|
|
|
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
|
|
// See the License for the specific language governing permissions and
|
|
|
|
// limitations under the License.
|
|
|
|
|
2016-12-04 12:16:11 +00:00
|
|
|
package tsdb
|
|
|
|
|
2016-12-09 09:41:51 +00:00
|
|
|
import (
|
2017-03-26 18:10:12 +00:00
|
|
|
"encoding/binary"
|
2016-12-09 09:41:51 +00:00
|
|
|
"sort"
|
|
|
|
"strings"
|
2017-09-05 09:45:18 +00:00
|
|
|
"sync"
|
|
|
|
|
|
|
|
"github.com/prometheus/tsdb/labels"
|
2016-12-09 09:41:51 +00:00
|
|
|
)
|
2016-12-04 12:16:11 +00:00
|
|
|
|
2016-12-22 00:12:28 +00:00
|
|
|
type memPostings struct {
|
2017-09-05 09:45:18 +00:00
|
|
|
mtx sync.RWMutex
|
|
|
|
m map[labels.Label][]uint64
|
2016-12-10 08:44:00 +00:00
|
|
|
}
|
|
|
|
|
2017-09-05 09:45:18 +00:00
|
|
|
func newMemPostings() *memPostings {
|
|
|
|
return &memPostings{
|
|
|
|
m: make(map[labels.Label][]uint64, 512),
|
|
|
|
}
|
2016-12-04 12:16:11 +00:00
|
|
|
}
|
|
|
|
|
2016-12-09 09:41:51 +00:00
|
|
|
// Postings returns an iterator over the postings list for s.
|
2017-09-05 09:45:18 +00:00
|
|
|
func (p *memPostings) get(name, value string) Postings {
|
|
|
|
p.mtx.RLock()
|
|
|
|
l := p.m[labels.Label{Name: name, Value: value}]
|
|
|
|
p.mtx.RUnlock()
|
|
|
|
|
2016-12-31 09:19:02 +00:00
|
|
|
if l == nil {
|
|
|
|
return emptyPostings
|
|
|
|
}
|
2017-04-23 23:53:56 +00:00
|
|
|
return newListPostings(l)
|
2016-12-09 09:41:51 +00:00
|
|
|
}
|
|
|
|
|
2017-09-20 16:08:57 +00:00
|
|
|
var allPostingsKey = labels.Label{}
|
2017-09-05 09:45:18 +00:00
|
|
|
|
2016-12-09 09:41:51 +00:00
|
|
|
// add adds a document to the index. The caller has to ensure that no
|
|
|
|
// term argument appears twice.
|
2017-09-05 09:45:18 +00:00
|
|
|
func (p *memPostings) add(id uint64, lset labels.Labels) {
|
|
|
|
p.mtx.Lock()
|
|
|
|
|
|
|
|
for _, l := range lset {
|
2017-09-20 16:08:57 +00:00
|
|
|
p.addFor(id, l)
|
2016-12-09 09:41:51 +00:00
|
|
|
}
|
2017-09-20 16:08:57 +00:00
|
|
|
p.addFor(id, allPostingsKey)
|
2017-09-05 09:45:18 +00:00
|
|
|
|
|
|
|
p.mtx.Unlock()
|
2016-12-04 12:16:11 +00:00
|
|
|
}
|
|
|
|
|
2017-09-20 16:08:57 +00:00
|
|
|
func (p *memPostings) addFor(id uint64, l labels.Label) {
|
|
|
|
list := append(p.m[l], id)
|
|
|
|
p.m[l] = list
|
|
|
|
|
|
|
|
// There is no guarantee that no higher ID was inserted before as they may
|
|
|
|
// be generated independently before adding them to postings.
|
|
|
|
// We repair order violations on insert. The invariant is that the first n-1
|
|
|
|
// items in the list are already sorted.
|
|
|
|
for i := len(list) - 1; i >= 1; i-- {
|
|
|
|
if list[i] >= list[i-1] {
|
|
|
|
break
|
|
|
|
}
|
|
|
|
list[i], list[i-1] = list[i-1], list[i]
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
func expandPostings(p Postings) (res []uint64, err error) {
|
|
|
|
for p.Next() {
|
|
|
|
res = append(res, p.At())
|
|
|
|
}
|
|
|
|
return res, p.Err()
|
|
|
|
}
|
|
|
|
|
2016-12-14 20:58:29 +00:00
|
|
|
// Postings provides iterative access over a postings list.
|
|
|
|
type Postings interface {
|
2016-12-13 14:26:58 +00:00
|
|
|
// Next advances the iterator and returns true if another value was found.
|
2016-12-04 12:16:11 +00:00
|
|
|
Next() bool
|
2016-12-13 14:26:58 +00:00
|
|
|
|
2016-12-04 12:16:11 +00:00
|
|
|
// Seek advances the iterator to value v or greater and returns
|
|
|
|
// true if a value was found.
|
2017-09-04 14:08:38 +00:00
|
|
|
Seek(v uint64) bool
|
2016-12-13 14:26:58 +00:00
|
|
|
|
2017-01-02 12:27:52 +00:00
|
|
|
// At returns the value at the current iterator position.
|
2017-09-04 14:08:38 +00:00
|
|
|
At() uint64
|
2016-12-13 14:26:58 +00:00
|
|
|
|
|
|
|
// Err returns the last error of the iterator.
|
|
|
|
Err() error
|
2016-12-04 12:16:11 +00:00
|
|
|
}
|
|
|
|
|
2016-12-14 20:58:29 +00:00
|
|
|
// errPostings is an empty iterator that always errors.
|
|
|
|
type errPostings struct {
|
2016-12-13 14:26:58 +00:00
|
|
|
err error
|
|
|
|
}
|
|
|
|
|
2016-12-14 20:58:29 +00:00
|
|
|
func (e errPostings) Next() bool { return false }
|
2017-09-04 14:08:38 +00:00
|
|
|
func (e errPostings) Seek(uint64) bool { return false }
|
|
|
|
func (e errPostings) At() uint64 { return 0 }
|
2016-12-14 20:58:29 +00:00
|
|
|
func (e errPostings) Err() error { return e.err }
|
2016-12-13 14:26:58 +00:00
|
|
|
|
2017-04-23 23:53:56 +00:00
|
|
|
var emptyPostings = errPostings{}
|
2016-12-27 10:32:10 +00:00
|
|
|
|
2016-12-14 20:58:29 +00:00
|
|
|
// Intersect returns a new postings list over the intersection of the
|
|
|
|
// input postings.
|
|
|
|
func Intersect(its ...Postings) Postings {
|
2016-12-04 12:16:11 +00:00
|
|
|
if len(its) == 0 {
|
2017-04-23 23:53:56 +00:00
|
|
|
return emptyPostings
|
2016-12-04 12:16:11 +00:00
|
|
|
}
|
2017-06-13 06:25:13 +00:00
|
|
|
if len(its) == 1 {
|
|
|
|
return its[0]
|
2016-12-04 12:16:11 +00:00
|
|
|
}
|
2017-06-13 06:25:13 +00:00
|
|
|
l := len(its) / 2
|
|
|
|
return newIntersectPostings(Intersect(its[:l]...), Intersect(its[l:]...))
|
2016-12-04 12:16:11 +00:00
|
|
|
}
|
|
|
|
|
2016-12-14 20:58:29 +00:00
|
|
|
type intersectPostings struct {
|
2016-12-27 10:32:10 +00:00
|
|
|
a, b Postings
|
|
|
|
aok, bok bool
|
2017-09-04 14:08:38 +00:00
|
|
|
cur uint64
|
2016-12-27 10:32:10 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
func newIntersectPostings(a, b Postings) *intersectPostings {
|
2017-04-23 23:53:56 +00:00
|
|
|
return &intersectPostings{a: a, b: b}
|
2016-12-04 12:16:11 +00:00
|
|
|
}
|
|
|
|
|
2017-09-04 14:08:38 +00:00
|
|
|
func (it *intersectPostings) At() uint64 {
|
2016-12-27 10:32:10 +00:00
|
|
|
return it.cur
|
2016-12-04 12:16:11 +00:00
|
|
|
}
|
|
|
|
|
2017-09-04 14:08:38 +00:00
|
|
|
func (it *intersectPostings) doNext(id uint64) bool {
|
2016-12-27 10:32:10 +00:00
|
|
|
for {
|
2017-04-23 23:53:56 +00:00
|
|
|
if !it.b.Seek(id) {
|
2016-12-27 10:32:10 +00:00
|
|
|
return false
|
|
|
|
}
|
2017-04-23 23:53:56 +00:00
|
|
|
if vb := it.b.At(); vb != id {
|
|
|
|
if !it.a.Seek(vb) {
|
|
|
|
return false
|
|
|
|
}
|
|
|
|
id = it.a.At()
|
|
|
|
if vb != id {
|
|
|
|
continue
|
|
|
|
}
|
2016-12-27 10:32:10 +00:00
|
|
|
}
|
2017-04-23 23:53:56 +00:00
|
|
|
it.cur = id
|
|
|
|
return true
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
func (it *intersectPostings) Next() bool {
|
|
|
|
if !it.a.Next() {
|
|
|
|
return false
|
2016-12-27 10:32:10 +00:00
|
|
|
}
|
2017-04-23 23:53:56 +00:00
|
|
|
return it.doNext(it.a.At())
|
2016-12-04 12:16:11 +00:00
|
|
|
}
|
|
|
|
|
2017-09-04 14:08:38 +00:00
|
|
|
func (it *intersectPostings) Seek(id uint64) bool {
|
2017-04-23 23:53:56 +00:00
|
|
|
if !it.a.Seek(id) {
|
|
|
|
return false
|
|
|
|
}
|
|
|
|
return it.doNext(it.a.At())
|
2016-12-04 12:16:11 +00:00
|
|
|
}
|
|
|
|
|
2016-12-14 20:58:29 +00:00
|
|
|
func (it *intersectPostings) Err() error {
|
2016-12-27 10:32:10 +00:00
|
|
|
if it.a.Err() != nil {
|
|
|
|
return it.a.Err()
|
|
|
|
}
|
|
|
|
return it.b.Err()
|
2016-12-13 14:26:58 +00:00
|
|
|
}
|
|
|
|
|
2016-12-04 12:16:11 +00:00
|
|
|
// Merge returns a new iterator over the union of the input iterators.
|
2016-12-14 20:58:29 +00:00
|
|
|
func Merge(its ...Postings) Postings {
|
2016-12-04 12:16:11 +00:00
|
|
|
if len(its) == 0 {
|
|
|
|
return nil
|
|
|
|
}
|
2017-06-13 06:25:13 +00:00
|
|
|
if len(its) == 1 {
|
|
|
|
return its[0]
|
2016-12-04 12:16:11 +00:00
|
|
|
}
|
2017-06-13 06:25:13 +00:00
|
|
|
l := len(its) / 2
|
|
|
|
return newMergedPostings(Merge(its[:l]...), Merge(its[l:]...))
|
2016-12-04 12:16:11 +00:00
|
|
|
}
|
|
|
|
|
2017-04-08 15:12:29 +00:00
|
|
|
type mergedPostings struct {
|
2017-04-23 23:53:56 +00:00
|
|
|
a, b Postings
|
|
|
|
initialized bool
|
|
|
|
aok, bok bool
|
2017-09-04 14:08:38 +00:00
|
|
|
cur uint64
|
2016-12-28 10:02:19 +00:00
|
|
|
}
|
|
|
|
|
2017-04-08 15:12:29 +00:00
|
|
|
func newMergedPostings(a, b Postings) *mergedPostings {
|
2017-04-23 23:53:56 +00:00
|
|
|
return &mergedPostings{a: a, b: b}
|
2016-12-04 12:16:11 +00:00
|
|
|
}
|
|
|
|
|
2017-09-04 14:08:38 +00:00
|
|
|
func (it *mergedPostings) At() uint64 {
|
2016-12-28 10:02:19 +00:00
|
|
|
return it.cur
|
2016-12-04 12:16:11 +00:00
|
|
|
}
|
|
|
|
|
2017-04-08 15:12:29 +00:00
|
|
|
func (it *mergedPostings) Next() bool {
|
2017-04-23 23:53:56 +00:00
|
|
|
if !it.initialized {
|
|
|
|
it.aok = it.a.Next()
|
|
|
|
it.bok = it.b.Next()
|
|
|
|
it.initialized = true
|
|
|
|
}
|
|
|
|
|
2016-12-28 10:02:19 +00:00
|
|
|
if !it.aok && !it.bok {
|
|
|
|
return false
|
|
|
|
}
|
|
|
|
|
|
|
|
if !it.aok {
|
2017-01-02 12:27:52 +00:00
|
|
|
it.cur = it.b.At()
|
2016-12-28 10:02:19 +00:00
|
|
|
it.bok = it.b.Next()
|
|
|
|
return true
|
|
|
|
}
|
|
|
|
if !it.bok {
|
2017-01-02 12:27:52 +00:00
|
|
|
it.cur = it.a.At()
|
2016-12-28 10:02:19 +00:00
|
|
|
it.aok = it.a.Next()
|
|
|
|
return true
|
|
|
|
}
|
|
|
|
|
2017-01-02 12:27:52 +00:00
|
|
|
acur, bcur := it.a.At(), it.b.At()
|
2016-12-28 10:02:19 +00:00
|
|
|
|
|
|
|
if acur < bcur {
|
|
|
|
it.cur = acur
|
|
|
|
it.aok = it.a.Next()
|
2017-04-23 23:53:56 +00:00
|
|
|
} else if acur > bcur {
|
2016-12-28 10:02:19 +00:00
|
|
|
it.cur = bcur
|
|
|
|
it.bok = it.b.Next()
|
2017-04-23 23:53:56 +00:00
|
|
|
} else {
|
|
|
|
it.cur = acur
|
|
|
|
it.aok = it.a.Next()
|
|
|
|
it.bok = it.b.Next()
|
2016-12-28 10:02:19 +00:00
|
|
|
}
|
|
|
|
return true
|
2016-12-04 12:16:11 +00:00
|
|
|
}
|
|
|
|
|
2017-09-04 14:08:38 +00:00
|
|
|
func (it *mergedPostings) Seek(id uint64) bool {
|
Fix missing postings in Merge and Intersect (#77)
* Test for a previous implematation of Intersect
Before we were moving the postings list everytime we create a new
chained `intersectPostings`. That was causing some postings to be
skipped. This test fails on the older version.
Signed-off-by: Goutham Veeramachaneni <cs14btech11014@iith.ac.in>
* Advance on Seek only when valid.
Issue:
Before in mergedPostings and others we advance everytime we `Seek`,
which causes issues with `Intersect`.
Take the case, where we have a mergedPostings = m merging, a: {10, 20, 30} and
b: {15, 25, 35}. Everytime we `Seek`, we do a.Seek and b.Seek.
Now if we Intersect m with {21, 22, 23, 30}, we would do Seek({21,22,23}) which
would advance a and b beyond 30.
Fix:
Now we advance only when the seeking value is greater than the current
value, as the definition specifies.
Also, posting 0 will not be a valid posting and will be used to signal
finished or un-initialized PostingsList.
Signed-off-by: Goutham Veeramachaneni <cs14btech11014@iith.ac.in>
* Add test for Merge+Intersect edgecase.
Signed-off-by: Goutham Veeramachaneni <cs14btech11014@iith.ac.in>
* Add comments to trivial tests.
Signed-off-by: Goutham Veeramachaneni <cs14btech11014@iith.ac.in>
2017-05-12 07:44:41 +00:00
|
|
|
if it.cur >= id {
|
|
|
|
return true
|
|
|
|
}
|
|
|
|
|
2016-12-28 10:02:19 +00:00
|
|
|
it.aok = it.a.Seek(id)
|
|
|
|
it.bok = it.b.Seek(id)
|
2017-04-23 23:53:56 +00:00
|
|
|
it.initialized = true
|
Fix missing postings in Merge and Intersect (#77)
* Test for a previous implematation of Intersect
Before we were moving the postings list everytime we create a new
chained `intersectPostings`. That was causing some postings to be
skipped. This test fails on the older version.
Signed-off-by: Goutham Veeramachaneni <cs14btech11014@iith.ac.in>
* Advance on Seek only when valid.
Issue:
Before in mergedPostings and others we advance everytime we `Seek`,
which causes issues with `Intersect`.
Take the case, where we have a mergedPostings = m merging, a: {10, 20, 30} and
b: {15, 25, 35}. Everytime we `Seek`, we do a.Seek and b.Seek.
Now if we Intersect m with {21, 22, 23, 30}, we would do Seek({21,22,23}) which
would advance a and b beyond 30.
Fix:
Now we advance only when the seeking value is greater than the current
value, as the definition specifies.
Also, posting 0 will not be a valid posting and will be used to signal
finished or un-initialized PostingsList.
Signed-off-by: Goutham Veeramachaneni <cs14btech11014@iith.ac.in>
* Add test for Merge+Intersect edgecase.
Signed-off-by: Goutham Veeramachaneni <cs14btech11014@iith.ac.in>
* Add comments to trivial tests.
Signed-off-by: Goutham Veeramachaneni <cs14btech11014@iith.ac.in>
2017-05-12 07:44:41 +00:00
|
|
|
|
|
|
|
return it.Next()
|
2016-12-04 12:16:11 +00:00
|
|
|
}
|
|
|
|
|
2017-04-08 15:12:29 +00:00
|
|
|
func (it *mergedPostings) Err() error {
|
2016-12-28 10:02:19 +00:00
|
|
|
if it.a.Err() != nil {
|
|
|
|
return it.a.Err()
|
|
|
|
}
|
|
|
|
return it.b.Err()
|
2016-12-13 14:26:58 +00:00
|
|
|
}
|
|
|
|
|
2016-12-19 10:44:11 +00:00
|
|
|
// listPostings implements the Postings interface over a plain list.
|
|
|
|
type listPostings struct {
|
2017-09-04 14:08:38 +00:00
|
|
|
list []uint64
|
|
|
|
cur uint64
|
2016-12-04 12:16:11 +00:00
|
|
|
}
|
|
|
|
|
2017-09-04 14:08:38 +00:00
|
|
|
func newListPostings(list []uint64) *listPostings {
|
2017-04-23 23:53:56 +00:00
|
|
|
return &listPostings{list: list}
|
2016-12-28 07:50:20 +00:00
|
|
|
}
|
|
|
|
|
2017-09-04 14:08:38 +00:00
|
|
|
func (it *listPostings) At() uint64 {
|
2017-04-23 23:53:56 +00:00
|
|
|
return it.cur
|
2016-12-04 12:16:11 +00:00
|
|
|
}
|
|
|
|
|
2016-12-19 10:44:11 +00:00
|
|
|
func (it *listPostings) Next() bool {
|
2017-04-23 23:53:56 +00:00
|
|
|
if len(it.list) > 0 {
|
|
|
|
it.cur = it.list[0]
|
|
|
|
it.list = it.list[1:]
|
|
|
|
return true
|
|
|
|
}
|
Fix missing postings in Merge and Intersect (#77)
* Test for a previous implematation of Intersect
Before we were moving the postings list everytime we create a new
chained `intersectPostings`. That was causing some postings to be
skipped. This test fails on the older version.
Signed-off-by: Goutham Veeramachaneni <cs14btech11014@iith.ac.in>
* Advance on Seek only when valid.
Issue:
Before in mergedPostings and others we advance everytime we `Seek`,
which causes issues with `Intersect`.
Take the case, where we have a mergedPostings = m merging, a: {10, 20, 30} and
b: {15, 25, 35}. Everytime we `Seek`, we do a.Seek and b.Seek.
Now if we Intersect m with {21, 22, 23, 30}, we would do Seek({21,22,23}) which
would advance a and b beyond 30.
Fix:
Now we advance only when the seeking value is greater than the current
value, as the definition specifies.
Also, posting 0 will not be a valid posting and will be used to signal
finished or un-initialized PostingsList.
Signed-off-by: Goutham Veeramachaneni <cs14btech11014@iith.ac.in>
* Add test for Merge+Intersect edgecase.
Signed-off-by: Goutham Veeramachaneni <cs14btech11014@iith.ac.in>
* Add comments to trivial tests.
Signed-off-by: Goutham Veeramachaneni <cs14btech11014@iith.ac.in>
2017-05-12 07:44:41 +00:00
|
|
|
it.cur = 0
|
2017-04-23 23:53:56 +00:00
|
|
|
return false
|
2016-12-04 12:16:11 +00:00
|
|
|
}
|
|
|
|
|
2017-09-04 14:08:38 +00:00
|
|
|
func (it *listPostings) Seek(x uint64) bool {
|
Fix missing postings in Merge and Intersect (#77)
* Test for a previous implematation of Intersect
Before we were moving the postings list everytime we create a new
chained `intersectPostings`. That was causing some postings to be
skipped. This test fails on the older version.
Signed-off-by: Goutham Veeramachaneni <cs14btech11014@iith.ac.in>
* Advance on Seek only when valid.
Issue:
Before in mergedPostings and others we advance everytime we `Seek`,
which causes issues with `Intersect`.
Take the case, where we have a mergedPostings = m merging, a: {10, 20, 30} and
b: {15, 25, 35}. Everytime we `Seek`, we do a.Seek and b.Seek.
Now if we Intersect m with {21, 22, 23, 30}, we would do Seek({21,22,23}) which
would advance a and b beyond 30.
Fix:
Now we advance only when the seeking value is greater than the current
value, as the definition specifies.
Also, posting 0 will not be a valid posting and will be used to signal
finished or un-initialized PostingsList.
Signed-off-by: Goutham Veeramachaneni <cs14btech11014@iith.ac.in>
* Add test for Merge+Intersect edgecase.
Signed-off-by: Goutham Veeramachaneni <cs14btech11014@iith.ac.in>
* Add comments to trivial tests.
Signed-off-by: Goutham Veeramachaneni <cs14btech11014@iith.ac.in>
2017-05-12 07:44:41 +00:00
|
|
|
// If the current value satisfies, then return.
|
|
|
|
if it.cur >= x {
|
|
|
|
return true
|
|
|
|
}
|
|
|
|
|
2016-12-04 12:16:11 +00:00
|
|
|
// Do binary search between current position and end.
|
2017-04-23 23:53:56 +00:00
|
|
|
i := sort.Search(len(it.list), func(i int) bool {
|
|
|
|
return it.list[i] >= x
|
2016-12-04 12:16:11 +00:00
|
|
|
})
|
2017-04-23 23:53:56 +00:00
|
|
|
if i < len(it.list) {
|
|
|
|
it.cur = it.list[i]
|
|
|
|
it.list = it.list[i+1:]
|
|
|
|
return true
|
|
|
|
}
|
|
|
|
it.list = nil
|
|
|
|
return false
|
2016-12-04 12:16:11 +00:00
|
|
|
}
|
2016-12-09 09:41:51 +00:00
|
|
|
|
2016-12-19 10:44:11 +00:00
|
|
|
func (it *listPostings) Err() error {
|
2016-12-13 14:26:58 +00:00
|
|
|
return nil
|
|
|
|
}
|
|
|
|
|
2017-03-27 08:34:42 +00:00
|
|
|
// bigEndianPostings implements the Postings interface over a byte stream of
|
|
|
|
// big endian numbers.
|
|
|
|
type bigEndianPostings struct {
|
2017-03-26 18:10:12 +00:00
|
|
|
list []byte
|
2017-04-23 23:53:56 +00:00
|
|
|
cur uint32
|
2017-03-26 18:10:12 +00:00
|
|
|
}
|
|
|
|
|
2017-03-27 08:34:42 +00:00
|
|
|
func newBigEndianPostings(list []byte) *bigEndianPostings {
|
2017-04-23 23:53:56 +00:00
|
|
|
return &bigEndianPostings{list: list}
|
2017-03-26 18:10:12 +00:00
|
|
|
}
|
|
|
|
|
2017-09-04 14:08:38 +00:00
|
|
|
func (it *bigEndianPostings) At() uint64 {
|
|
|
|
return uint64(it.cur)
|
2017-03-26 18:10:12 +00:00
|
|
|
}
|
|
|
|
|
2017-03-27 08:34:42 +00:00
|
|
|
func (it *bigEndianPostings) Next() bool {
|
2017-04-23 23:53:56 +00:00
|
|
|
if len(it.list) >= 4 {
|
|
|
|
it.cur = binary.BigEndian.Uint32(it.list)
|
|
|
|
it.list = it.list[4:]
|
|
|
|
return true
|
|
|
|
}
|
|
|
|
return false
|
2017-03-26 18:10:12 +00:00
|
|
|
}
|
|
|
|
|
2017-09-04 14:08:38 +00:00
|
|
|
func (it *bigEndianPostings) Seek(x uint64) bool {
|
|
|
|
if uint64(it.cur) >= x {
|
Fix missing postings in Merge and Intersect (#77)
* Test for a previous implematation of Intersect
Before we were moving the postings list everytime we create a new
chained `intersectPostings`. That was causing some postings to be
skipped. This test fails on the older version.
Signed-off-by: Goutham Veeramachaneni <cs14btech11014@iith.ac.in>
* Advance on Seek only when valid.
Issue:
Before in mergedPostings and others we advance everytime we `Seek`,
which causes issues with `Intersect`.
Take the case, where we have a mergedPostings = m merging, a: {10, 20, 30} and
b: {15, 25, 35}. Everytime we `Seek`, we do a.Seek and b.Seek.
Now if we Intersect m with {21, 22, 23, 30}, we would do Seek({21,22,23}) which
would advance a and b beyond 30.
Fix:
Now we advance only when the seeking value is greater than the current
value, as the definition specifies.
Also, posting 0 will not be a valid posting and will be used to signal
finished or un-initialized PostingsList.
Signed-off-by: Goutham Veeramachaneni <cs14btech11014@iith.ac.in>
* Add test for Merge+Intersect edgecase.
Signed-off-by: Goutham Veeramachaneni <cs14btech11014@iith.ac.in>
* Add comments to trivial tests.
Signed-off-by: Goutham Veeramachaneni <cs14btech11014@iith.ac.in>
2017-05-12 07:44:41 +00:00
|
|
|
return true
|
|
|
|
}
|
|
|
|
|
2017-03-26 18:10:12 +00:00
|
|
|
num := len(it.list) / 4
|
|
|
|
// Do binary search between current position and end.
|
2017-04-23 23:53:56 +00:00
|
|
|
i := sort.Search(num, func(i int) bool {
|
2017-09-04 14:08:38 +00:00
|
|
|
return binary.BigEndian.Uint32(it.list[i*4:]) >= uint32(x)
|
2017-03-26 18:10:12 +00:00
|
|
|
})
|
2017-04-23 23:53:56 +00:00
|
|
|
if i < num {
|
|
|
|
j := i * 4
|
|
|
|
it.cur = binary.BigEndian.Uint32(it.list[j:])
|
|
|
|
it.list = it.list[j+4:]
|
|
|
|
return true
|
|
|
|
}
|
|
|
|
it.list = nil
|
|
|
|
return false
|
2017-03-26 18:10:12 +00:00
|
|
|
}
|
|
|
|
|
2017-03-27 08:34:42 +00:00
|
|
|
func (it *bigEndianPostings) Err() error {
|
2017-03-26 18:10:12 +00:00
|
|
|
return nil
|
|
|
|
}
|
|
|
|
|
2016-12-09 09:41:51 +00:00
|
|
|
type stringset map[string]struct{}
|
|
|
|
|
|
|
|
func (ss stringset) set(s string) {
|
|
|
|
ss[s] = struct{}{}
|
|
|
|
}
|
|
|
|
|
|
|
|
func (ss stringset) has(s string) bool {
|
|
|
|
_, ok := ss[s]
|
|
|
|
return ok
|
|
|
|
}
|
|
|
|
|
|
|
|
func (ss stringset) String() string {
|
|
|
|
return strings.Join(ss.slice(), ",")
|
|
|
|
}
|
|
|
|
|
|
|
|
func (ss stringset) slice() []string {
|
|
|
|
slice := make([]string, 0, len(ss))
|
|
|
|
for k := range ss {
|
|
|
|
slice = append(slice, k)
|
|
|
|
}
|
|
|
|
sort.Strings(slice)
|
|
|
|
return slice
|
|
|
|
}
|