-
Notifications
You must be signed in to change notification settings - Fork 3.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
docs: describe ordering in http query results #6024
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍 thanks @gabor.
@KMiller-Grafana leave it to you for final approval
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
Co-authored-by: Karen Miller <84039272+KMiller-Grafana@users.noreply.github.com>
Co-authored-by: Karen Miller <84039272+KMiller-Grafana@users.noreply.github.com>
Co-authored-by: Karen Miller <84039272+KMiller-Grafana@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the changes.
this pull-request describes the ordering of results when using
/query
and/query_range
HTTP endpoints.it is important to know whether one can rely on the results being ordered or not (for example, in the grafana loki datasource, if we cannot rely on the ordering of results, we have to sort the data)
NOTE: i do not know if what i described here is guaranteed or not. i made some tests, and it seems to work this way. it would be great if a Loki developer could also verify this info. (if it is not guaranteed, i can update the pull-request so that the docs will mention that the ordering is not guaranteed)
also, there was an error in the query-range-matrix-results format description, i fixed that.