-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
Update cortex vendoring to latest master #1975
Conversation
@@ -168,6 +169,13 @@ func (shard *indexShard) lookup(matchers []*labels.Matcher) []model.Fingerprint | |||
if matcher.Type == labels.MatchEqual { | |||
fps := values.fps[matcher.Value] | |||
toIntersect = append(toIntersect, fps.fps...) // deliberate copy | |||
} else if matcher.Type == labels.MatchRegexp && len(chunk.FindSetMatches(matcher.Value)) > 0 { |
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.
Nice I needed this, and I was bringing it in another PR.
@sandeepsukhani It might be easier for you to fix this than me? Looks like the |
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.
I just went through all the changes and fixed the ones which were breaking.
Looks good to go to me unless someone else wants to have a look at it to reconfirm.
Thanks @sandeepsukhani ! |
…d7c74136cab1566fbd9583` Signed-off-by: Ed Welch <edward.welch@grafana.com>
Signed-off-by: Sandeep Sukhani <sandeep.d.sukhani@gmail.com>
…ore changes... Signed-off-by: Ed Welch <edward.welch@grafana.com>
97134a4
to
3df3274
Compare
Commit hash:
36496a074bc4530fb1d7c74136cab1566fbd9583
Signed-off-by: Ed Welch edward.welch@grafana.com