-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
fix descending and range queries via IFQL RPC API #9105
Conversation
@@ -0,0 +1 @@ | |||
package tsm1 |
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.
Empty test file?
tsdb/engine/tsm1/engine_test.go
Outdated
} | ||
} | ||
|
||
// Ensure engine can create an ascending cursor for tsm values. |
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.
descending cursor in this test
CHANGELOG.md
Outdated
@@ -1,3 +1,9 @@ | |||
## v1.4.1 [unreleased] |
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.
Since we're just doing a quick patch release for this, can you go ahead and set the date to 2017-11-13?
3b99381
to
3ba39a1
Compare
* did not handle cached values correctly * sort shards by time in either ascending or descending order depending on the RPC request ordering to ensure they are traversed in the correct order. (cherry picked from commit 2e04e87) 4039a76 Stuart Carnie <stuart.carnie@gmail.com> on 11/13/17 at 4:07 PM committed on 11/13/17 at 5:22 PM
3ba39a1
to
5d4f570
Compare
order depending on the RPC request ordering to ensure they
are traversed in the correct order.
(cherry picked from commit 2e04e87)
Required for all non-trivial PRs