go-graphite / carbonapi

Implementation of graphite API (graphite-web) in golang
Other
309 stars 140 forks source link

Fix consolidateFunc setting and add test for consolidateBy #733

Closed carrieedwards closed 1 year ago

carrieedwards commented 1 year ago

This PR adds tests for the consolidateBy function, as there previously were no tests. It also updates the Name field of the results to match how Graphite web's consolidateBy function assigns Name to the results (see Graphite web's implementation).

Additionally, it appeared that several functions, such as constantLine, time, and identity, had a consolidationFunc setting that was not aligned with Graphite web, so this was fixed.