Cool.  Just want to understand the type a little bit better.


2013/2/28 juliusv <notifications@github.com>

> In retrieval/targetmanager.go:
>
> > @@ -52,17 +53,32 @@ func (m *targetManager) release() {
> >  	<-m.requestAllowance
> >  }
> >
> > -func (m *targetManager) Add(t Target) {
> > -	targetPool, ok := m.pools[t.Interval()]
> > +func (m *targetManager) GetTargetPoolForJob(job *config.JobConfig, defaultScrapeInterval time.Duration) (targetPool *TargetPool) {
>
> Nah, I'm just not completely sold on the whole pointer-vs-value debate
> when passing structs. I'm just thinking, why would I want to make the
> processor/memory do extra work if I don't need to. Although that work is
> tiny, it just feels weird to my C-self ;)
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/prometheus/prometheus/pull/86/files#r3195207>
> .
>