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.

Sign up for GitHub

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

Jump to bottom

[r168] Wrap mutex around combiner now that results are processed in parallel #4154

Merged
merged 1 commit into from
Oct 3, 2024
Merged

[r168] Wrap mutex around combiner now that results are processed in parallel #4154

merged 1 commit into from
Oct 3, 2024

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Oct 3, 2024

Backport 90e9b11 from #4153

What this PR does:

Which issue(s) this PR fixes:
Fixes #

Checklist

  • Tests updated
  • Documentation added
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

@zalegrala @grafana-delivery-bot
Wrap mutex around combiner now that results are processed in parallel (…
f38ef25 
…#4153)

* Wrap mutex around combiner now that results are processed in parallel

* Add test for parallel combining

(cherry picked from commit 90e9b11)
@github-actions github-actions bot requested a review from joe-elliott as a code owner October 3, 2024 13:06
@github-actions github-actions bot added the type/bug Something isn't working label Oct 3, 2024
@yvrhdn yvrhdn closed this Oct 3, 2024
@yvrhdn yvrhdn reopened this Oct 3, 2024
@yvrhdn
Copy link
Member

yvrhdn commented Oct 3, 2024

Close/reopen to trigger Github Actions CI

@yvrhdn yvrhdn merged commit 69af9ca into r168 Oct 3, 2024
18 checks passed
@yvrhdn yvrhdn deleted the backport-4153-to-r168 branch October 3, 2024 13:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zalegrala zalegrala zalegrala approved these changes

@joe-elliott joe-elliott Awaiting requested review from joe-elliott joe-elliott is a code owner

@annanay25 annanay25 Awaiting requested review from annanay25 annanay25 is a code owner

@mdisibio mdisibio Awaiting requested review from mdisibio mdisibio is a code owner

@mapno mapno Awaiting requested review from mapno mapno is a code owner

@yvrhdn yvrhdn Awaiting requested review from yvrhdn

@electron0zero electron0zero Awaiting requested review from electron0zero electron0zero is a code owner

@ie-pham ie-pham Awaiting requested review from ie-pham ie-pham is a code owner

@stoewer stoewer Awaiting requested review from stoewer stoewer is a code owner

Assignees
No one assigned
Labels
backport type/bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@yvrhdn @zalegrala