Skip to content

[2.6] Add redisearch_thpool_wait in redisearch_thpool_destroy [MOD-10105] #6316

Merged
lerman25 merged 2 commits into2.6from
backport-6303-to-2.6
Jun 15, 2025
Merged

[2.6] Add redisearch_thpool_wait in redisearch_thpool_destroy [MOD-10105] #6316
lerman25 merged 2 commits into2.6from
backport-6303-to-2.6

Conversation

@lerman25
Copy link
Collaborator

Backport of #6303 to 2.6.

…6303)

* Add wait

* remove unnecessary check

(cherry picked from commit 1f3a9c5)
@codecov
Copy link

codecov bot commented Jun 15, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.82%. Comparing base (8d06105) to head (5b4b5bb).
Report is 2 commits behind head on 2.6.

Additional details and impacted files
@@            Coverage Diff             @@
##              2.6    #6316      +/-   ##
==========================================
- Coverage   84.86%   84.82%   -0.05%     
==========================================
  Files         189      189              
  Lines       31695    31693       -2     
==========================================
- Hits        26899    26884      -15     
- Misses       4796     4809      +13     
Flag Coverage Δ
flow 79.44% <ø> (-0.30%) ⬇️
unit 43.57% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@lerman25 lerman25 enabled auto-merge June 15, 2025 16:46
@lerman25 lerman25 added this pull request to the merge queue Jun 15, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jun 15, 2025
@lerman25 lerman25 added this pull request to the merge queue Jun 15, 2025
Merged via the queue into 2.6 with commit ad1183c Jun 15, 2025
10 checks passed
@lerman25 lerman25 deleted the backport-6303-to-2.6 branch June 15, 2025 21:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants