Skip to content

Commit c0e4545

Browse files
authored
stop using FromMetricsToLabelAdapters (#5078)
1 parent fe25f66 commit c0e4545

File tree

3 files changed

+9
-7
lines changed

3 files changed

+9
-7
lines changed

pkg/distributor/distributor.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1036,11 +1036,11 @@ func (d *Distributor) MetricsForLabelMatchers(ctx context.Context, from, through
10361036
if err := queryLimiter.AddDataBytes(resp.Size()); err != nil {
10371037
return nil, err
10381038
}
1039-
ms := ingester_client.FromMetricsForLabelMatchersResponse(resp)
1040-
for _, m := range ms {
1041-
if err := queryLimiter.AddSeries(cortexpb.FromMetricsToLabelAdapters(m)); err != nil {
1039+
for _, m := range resp.Metric {
1040+
if err := queryLimiter.AddSeries(m.Labels); err != nil {
10421041
return nil, err
10431042
}
1043+
m := cortexpb.FromLabelAdaptersToMetric(m.Labels)
10441044
fingerprint := m.Fingerprint()
10451045
mutex.Lock()
10461046
(*metrics)[fingerprint] = m

pkg/querier/tripperware/instantquery/instant_query.go

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@ import (
1717
otlog "github.com/opentracing/opentracing-go/log"
1818
"github.com/pkg/errors"
1919
"github.com/prometheus/common/model"
20+
"github.com/prometheus/prometheus/model/labels"
2021
"github.com/prometheus/prometheus/model/timestamp"
2122
"github.com/weaveworks/common/httpgrpc"
2223
"google.golang.org/grpc/status"
@@ -422,13 +423,13 @@ func decorateWithParamName(err error, field string) error {
422423
// UnmarshalJSON implements json.Unmarshaler.
423424
func (s *Sample) UnmarshalJSON(data []byte) error {
424425
var sample struct {
425-
Metric model.Metric `json:"metric"`
426+
Metric labels.Labels `json:"metric"`
426427
Value cortexpb.Sample `json:"value"`
427428
}
428429
if err := json.Unmarshal(data, &sample); err != nil {
429430
return err
430431
}
431-
s.Labels = cortexpb.FromMetricsToLabelAdapters(sample.Metric)
432+
s.Labels = cortexpb.FromLabelsToLabelAdapters(sample.Metric)
432433
s.Sample = sample.Value
433434
return nil
434435
}

pkg/querier/tripperware/query.go

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@ import (
1717
jsoniter "github.com/json-iterator/go"
1818
"github.com/opentracing/opentracing-go"
1919
"github.com/prometheus/common/model"
20+
"github.com/prometheus/prometheus/model/labels"
2021
"github.com/weaveworks/common/httpgrpc"
2122

2223
"github.com/cortexproject/cortex/pkg/cortexpb"
@@ -111,13 +112,13 @@ func encodeSampleStream(ptr unsafe.Pointer, stream *jsoniter.Stream) {
111112
// UnmarshalJSON implements json.Unmarshaler.
112113
func (s *SampleStream) UnmarshalJSON(data []byte) error {
113114
var stream struct {
114-
Metric model.Metric `json:"metric"`
115+
Metric labels.Labels `json:"metric"`
115116
Values []cortexpb.Sample `json:"values"`
116117
}
117118
if err := json.Unmarshal(data, &stream); err != nil {
118119
return err
119120
}
120-
s.Labels = cortexpb.FromMetricsToLabelAdapters(stream.Metric)
121+
s.Labels = cortexpb.FromLabelsToLabelAdapters(stream.Metric)
121122
s.Samples = stream.Values
122123
return nil
123124
}

0 commit comments

Comments
 (0)