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

MON-1116: Bump to f55d854c #28

Merged
merged 10 commits into from
Jun 4, 2020

Conversation

s-urbaniak
Copy link

This now also pulls in deduplication logic for Rules API

/cc @openshift/openshift-team-monitoring

goek-han and others added 9 commits June 3, 2020 10:20
Signed-off-by: Gökhan Kocak <[email protected]>

Co-authored-by: Gökhan Kocak <[email protected]>
* Added a placeholder for devdocs

Signed-off-by: Yash <[email protected]>

* some nitpickings

Signed-off-by: Yash <[email protected]>
* Added Series request hints support to BucketStore

Signed-off-by: Marco Pracucci <[email protected]>

* Refactoring based on feedback received

Signed-off-by: Marco Pracucci <[email protected]>

* Renamed var

Signed-off-by: Marco Pracucci <[email protected]>

* Improved tests

Signed-off-by: Marco Pracucci <[email protected]>

* Refactoring based on feedback received

Signed-off-by: Marco Pracucci <[email protected]>
* Improve binary index-header documentation

Signed-off-by: Marco Pracucci <[email protected]>

* Improved binary index-header doc based on feedback

Signed-off-by: Marco Pracucci <[email protected]>
* ui: Fix UI state being shared

Signed-off-by: Prem Kumar <[email protected]>

* format bindata.go

Signed-off-by: Prem Kumar <[email protected]>
…rting for StoreAPI + Optimized iter chunk dedup. (thanos-io#2710)

* Deduplicate chunk dups on proxy StoreAPI level. Recommend chunk sorting for StoreAPI.

Also: Merge same series together on proxy level instead select. This allows better dedup efficiency.

Partially fixes: thanos-io#2303

Cases like overlapped data from store and sidecar and 1:1 duplicates are optimized as soon as it's possible.
This case was highly visible on GitLab repro data and exists in most of Thanos setup.

Signed-off-by: Bartlomiej Plotka <[email protected]>

* Optimized algorithm to combine series only on start.

Signed-off-by: Bartlomiej Plotka <[email protected]>

* Optimized chunk comparision for overlaps.

Signed-off-by: Bartlomiej Plotka <[email protected]>

* Optimized deduplication for deduplicated chunk on query level as well.

Never use proto .String() in fast path!

Signed-off-by: Bartlomiej Plotka <[email protected]>
+ some nits from Bartek

Signed-off-by: Sergiusz Urbaniak <[email protected]>
@openshift-ci-robot openshift-ci-robot requested a review from a team June 4, 2020 10:11
@openshift-ci-robot openshift-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jun 4, 2020
@lilic
Copy link

lilic commented Jun 4, 2020

think it needs a revendor? Otherwise yay!! 🎉

@simonpasquier
Copy link

You need to update the vendor directory :)

@s-urbaniak
Copy link
Author

@lilic @simonpasquier indeed, i forgot about that one, ptal 👍

Copy link

@lilic lilic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm
🎉

@openshift-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: lilic, s-urbaniak

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Jun 4, 2020
@s-urbaniak
Copy link
Author

/test e2e-aws

@openshift-bot
Copy link

/retest

Please review the full test history for this PR and help us cut down flakes.

@s-urbaniak
Copy link
Author

/retest
so far everything unrelated

@s-urbaniak
Copy link
Author

/retest

@openshift-bot
Copy link

/retest

Please review the full test history for this PR and help us cut down flakes.

1 similar comment
@openshift-bot
Copy link

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-merge-robot openshift-merge-robot merged commit 64130f9 into openshift:master Jun 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.