filodb / FiloDB

Distributed Prometheus time series database
Apache License 2.0
1.43k stars 225 forks source link

fix(query): Reverting the min max suffix replacement #1757

Closed sandeep6189 closed 6 months ago

sandeep6189 commented 6 months ago

Pull Request checklist

Current behavior : Assertion error happening in cases where

  1. metric name has a suffix _min or _max AND
  2. query have lots of label filters which causes some TimeSeriesShard returning no data AND
  3. have another metric without the suffix.

New behavior : We are reverting the query change added with the support for min/max histograms to fix the bug in the short term.