I believe that is correct.  I would want to verify it, though.


2013/4/9 juliusv <notifications@github.com>

> In model/fingerprinting.go:
>
> > @@ -151,20 +152,7 @@ func (f fingerprint) LastCharacterOfLastLabelValue() string {
> >  }
> >
> >  func (f fingerprint) Less(o Fingerprint) bool {
> > -	if f.Hash() < o.Hash() {
> > -		return true
> > -	}
> > -	if f.FirstCharacterOfFirstLabelName() < o.FirstCharacterOfFirstLabelName() {
> > -		return true
> > -	}
> > -	if f.LabelMatterLength() < o.LabelMatterLength() {
> > -		return true
> > -	}
> > -	if f.LastCharacterOfLastLabelValue() < o.LastCharacterOfLastLabelValue() {
> > -		return true
> > -	}
> > -
> > -	return false
> > +	return f.String() < o.String()
>
> Question: String() is an alias for ToRowKey(), so even if "1000" < "2" by
> that definition, that ordering is still consistent with what's on disk,
> right?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/prometheus/prometheus/pull/121/files#r3710121>
> .
>