Fix updating of NumTombstones in block.Delete(..)

[codesome]

Fixes #384

Signed-off-by: Ganesh Vernekar [email protected]

[krasi-georgiev]

I would say something like this would be a bit more clear and better to avoid races:

// TombstoneReader gives access to tombstone intervals by series reference.
type TombstoneReader interface {
	...
        // Total returns the total count of Tombstones.
	Total() uint64
}


func (t *memTombstones) Total() uint64 {
	t.mtx.RLock()
	defer t.mtx.RUnlock()

	var total uint64
	for _, intvs := range t.intvlGroups {
		total += uint64(len(intvs))
	}
	return total
}


func (pb *Block) Delete
     ....
     pb.meta.Stats.NumTombstones = pb.tombstones.Total()

[codesome]

There wont be any race condition in the current change, but what you proposed looks good. I will add that function.

[krasi-georgiev]

LGTM

ping @gouthamve

[codesome]

ping @gouthamve @krasi-georgiev

[krasi-georgiev]

Thanks!