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
Make log query results cacheable #4095
Conversation
to splitting so that each intervals query can be cached. Signed-off-by: Callum Styan <callumstyan@gmail.com>
Signed-off-by: Callum Styan <callumstyan@gmail.com>
alignment logic into the splitter for that query type. Signed-off-by: Callum Styan <callumstyan@gmail.com>
I'm still interested into this. Adding the keepalive tag. |
Signed-off-by: Callum Styan <callumstyan@gmail.com>
Drone build seems to be stuck and I don't have admin access for this repo. |
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
Signed-off-by: Callum Styan <callumstyan@gmail.com>
I think we should do this for all query types. @cstyan do you mind if I push to your branch or open a new PR which would also cover your changes? |
Closing this as no activities for long time. Feel free to send new PR if anyone want's to revive the work |
This PR aligns a queries time range to multiples of the split-by interval prior to splitting so that each intervals query result can be cached. Note that the first and last interval are later changed again to use the original start and end time of the query so that we're not.
Signed-off-by: Callum Styan callumstyan@gmail.com