Definitely not keeping it.  :-)


2013/3/6 juliusv <notifications@github.com>

> In storage/metric/leveldb.go:
>
> > +	begin := time.Now()
> > +	defer func() {
> > +		duration := time.Now().Sub(begin)
> > +
> > +		recordOutcome(duration, err, map[string]string{operation: appendSample, result: success}, map[string]string{operation: appendSample, result: failure})
> > +	}()
> > +
> > +	err = l.AppendSamples(model.Samples{sample})
> > +
> > +	return
> > +}
> > +
> > +func (l *LevelDBMetricPersistence) AppendSamples(samples model.Samples) (err error) {
> > +	c := len(samples)
> > +	if c > 1 {
> > +		fmt.Printf("Appending %d samples...", c)
>
> "\n" at the end if you're keeping this Printf.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/prometheus/prometheus/pull/84/files#r3272980>
> .
>