Closed npazosmendez closed 2 years ago
We noticed some functions were setting the r.AggregateFunction directly instead of the string r.ConsolidationFunc that is then used in the GetAggregateFunction helper:
r.AggregateFunction
r.ConsolidationFunc
GetAggregateFunction
https://github.com/go-graphite/carbonapi/blob/30a090756e199c9d588499972268c4ebbb22a4e9/expr/types/types.go#L299-L310
This was leading to inconsistencies between ConsolidationFunc and AggregateFunction that were causing issues for our use case.
ConsolidationFunc
AggregateFunction
We noticed some functions were setting the
r.AggregateFunctiondirectly instead of the stringr.ConsolidationFuncthat is then used in theGetAggregateFunctionhelper:https://github.com/go-graphite/carbonapi/blob/30a090756e199c9d588499972268c4ebbb22a4e9/expr/types/types.go#L299-L310
This was leading to inconsistencies between
ConsolidationFuncandAggregateFunctionthat were causing issues for our use case.