You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Looks like go-kit/kit changed and I am getting errors when starting spigo?
go run spigo.go
# github.com/adrianco/spigo/tooling/collect
tooling/collect/collect.go:32: too many arguments in call to "github.com/go-kit/kit/metrics/expvar".NewHistogram
tooling/collect/collect.go:48: cannot use int64(maxHistObservable) (type int64) as type float64 in argument to h.Observe
tooling/collect/collect.go:50: cannot use int64(d) (type int64) as type float64 in argument to h.Observe
tooling/collect/collect.go:68: undefined: metrics.PrintDistribution
tooling/collect/collect.go:98: h.Name undefined (type metrics.Histogram has no field or method Name)
Maybe I am doing something wrong ?
The text was updated successfully, but these errors were encountered:
The metric interfaces are reduced to only what is necessary to make observations.
metrics.Field is removed. Use ordered string pairs, like package log, instead.
All metric values are float64. Thanks, Prometheus, for the inspiration.
TimeHistogram is removed. Prefer observing float64 seconds instead.
NewMulti{Histogram,Counter,Gauge} are moved to sub-package multi.
Minor changes:
PrintHistogram is removed. Users can use generic.Histogram.Print instead.
Distribution method is removed. Users can use generic.Histogram.Quantile instead.
Metrics systems that have similar emission semantics share similar architectures.
Metrics systems that support arbitrary label values share a basic implementation, lv.Space.
prometheus.RegisterCallbackGauge is gone. Use standard Prometheus NewGaugeFunc instead.
adrianco
changed the title
go run spigo.go
Spigo won't build due to go-kit/metrics refactor (was: go run spigo.go)
Sep 19, 2016
Looks like go-kit/kit changed and I am getting errors when starting spigo?
Maybe I am doing something wrong ?
The text was updated successfully, but these errors were encountered: