Skip to content

Commit

Permalink
changed interface name from Analyze to Analyzer in analyzer.go
Browse files Browse the repository at this point in the history
Signed-off-by: Rahul Kumar Jha <rahuljhakumar1600@gmail.com>
  • Loading branch information
Rahulkumar2002 committed Oct 12, 2022
1 parent 08674cc commit cbcff58
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions pkg/queryfrontend/shard_query.go
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ import (
)

// PromQLShardingMiddleware creates a new Middleware that shards PromQL aggregations using grouping labels.
func PromQLShardingMiddleware(queryAnalyzer querysharding.Analyze, numShards int, limits queryrange.Limits, merger queryrange.Merger, registerer prometheus.Registerer) queryrange.Middleware {
func PromQLShardingMiddleware(queryAnalyzer *querysharding.CachedQueryAnalyzer, numShards int, limits queryrange.Limits, merger queryrange.Merger, registerer prometheus.Registerer) queryrange.Middleware {
return queryrange.MiddlewareFunc(func(next queryrange.Handler) queryrange.Handler {
queriesTotal := promauto.With(registerer).NewCounterVec(prometheus.CounterOpts{
Namespace: "thanos",
Expand All @@ -45,7 +45,7 @@ type querySharder struct {
next queryrange.Handler
limits queryrange.Limits

queryAnalyzer querysharding.Analyze
queryAnalyzer *querysharding.CachedQueryAnalyzer
numShards int
merger queryrange.Merger

Expand Down
2 changes: 1 addition & 1 deletion pkg/querysharding/analyzer.go
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ import (
// QueryAnalyzer is an analyzer which determines
// whether a PromQL Query is shardable and using which labels.

type Analyze interface {
type Analyzer interface {
Analyze(string) (QueryAnalysis, error)
}

Expand Down

0 comments on commit cbcff58

Please sign in to comment.