*: rework Explain()/Analyze() name generation

thanos-io / promql-engine

Multi-threaded PromQL engine implementation based on the Volcano paper.

Apache License 2.0

131 stars 52 forks source link

*: rework Explain()/Analyze() name generation #421

Closed GiedriusS closed 3 months ago

GiedriusS commented 3 months ago

Currently the operator descriptions in Explain/Analyze are disconnected. We want for operators to look the same no matter if Explain or Analyze are used. Hence, add support for fmt.Stringer everywhere and use it.

fpetkovski commented 3 months ago

Looks good to me, we just need to fix CI.

GiedriusS commented 3 months ago

@fpetkovski should be good to go now