Bryan Boreham

Hello from the bug-scrub: @dgl do you have any further thoughts?

Still got CI errors - will you take another look at those?

> Running the benchmarks We did run prombench, although it doesn't have this kind of query. I don't think there are any benchmarks which currently test the case of an...

> I couldn't find it documented anywhere. Perhaps [here](https://prometheus.io/docs/prometheus/latest/querying/api/#range-vectors) makes sense? :+1: > Should we also require instant vectors to be sorted by series as well? The code takes care...

Related: the code silently ignores any `sort` in the PromQL when returning a matrix. In my mind I imagine some heartless Prometheus maintainer gloating "they'll figure it out eventually", but...

Also related: [`sort_by_label`](https://prometheus.io/docs/prometheus/latest/querying/functions/#sort_by_label) could work for matrixes.

Does it need some test cases? Are you sure that steps in the result set are always numbered the same as steps in the input to `rangeEval` and `eval` ?

@austinlparker sorry to bug you personally, but you were the last person to merge anything in this repo. Would you be interested in handing over to someone else?

Update: I have rebased, fixed up the things which changed due to native histograms ~, and added one more commit which makes a big improvement in `BenchmarkQuerierSelect`~. I would appreciate...

Hello from the bug-scrub! We believe this behaviour still exists, but we question whether this is something that Prometheus should deal with. Production systems should have clocks that are synchronized...