-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
query: Support chain deduplication algorithm #7808
base: main
Are you sure you want to change the base?
Conversation
You can run I wonder if we want to add a small paragraph in the query docs that explains when to use which dedup algorithm. |
73ce145
to
ed4edbc
Compare
Can I somehow rerun e2e tests? They worked before the rebase and also locally on my laptop, so I suspect that fails are due to flaky tests. |
It seems to consistently fail on other PRs too, so it's unrelated to this change. |
Signed-off-by: Michał Mazur <[email protected]>
Signed-off-by: Michał Mazur <[email protected]>
ed4edbc
to
04502d7
Compare
Sorry for nudging again, but I'm lost how to handle flakes here (these are my first PRs to thanos). |
I bumped the CI and everything is green now. I'll leave some time for others to take a look before merging. |
Changes
Adds --deduplication.func flag to query command.
All tests should work properly, however this PR isn't properly polished, yet.