Skip to content
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

Implement offset modifier for range vector aggregation in LogQL #3452

Closed
wants to merge 1 commit into from

Conversation

garrettlish
Copy link
Contributor

@garrettlish garrettlish commented Mar 9, 2021

implementation for #2785, the offset modifier allows changing the time offset for range vectors in a query to support e.g. selective timeshift in Grafana

Signed-off-by: garrettlish garrettlish@163.com

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


xiancli seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

implementation for grafana#2785, the offset modifier allows changing the time offset for range vectors in a query to support e.g. selective timeshift in Grafana

Signed-off-by: garrettlish <garrettlish@163.com>
@garrettlish
Copy link
Contributor Author

looks input with wrong user name, let me close this one and open a new one

@garrettlish garrettlish closed this Mar 9, 2021
cyriltovena pushed a commit to cyriltovena/loki that referenced this pull request Jun 11, 2021
…uture (grafana#3452)

* Added querier max-query-lookback and fixed max-query-into-future

Signed-off-by: Marco Pracucci <marco@pracucci.com>

* Updated PR number

Signed-off-by: Marco Pracucci <marco@pracucci.com>

* Fixed tests

Signed-off-by: Marco Pracucci <marco@pracucci.com>

* Removed dead code

Signed-off-by: Marco Pracucci <marco@pracucci.com>

* Added warning for the deprecated flag

Signed-off-by: Marco Pracucci <marco@pracucci.com>

* Addressed review comments

Signed-off-by: Marco Pracucci <marco@pracucci.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants