Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix thread pool scope #471

Merged
merged 1 commit into from
Mar 22, 2024
Merged

Conversation

MattFenelon
Copy link
Contributor

@MattFenelon MattFenelon commented Mar 22, 2024

The thread pool and mutex need to be global across instances of Scope for it to be a global thread pool.

See: #470

The thread pool and mutex need to be global across instances of Scope
for it to be a global thread pool.
@MattFenelon MattFenelon marked this pull request as ready for review March 22, 2024 10:46
@jkeen jkeen merged commit 51fb51c into graphiti-api:master Mar 22, 2024
36 checks passed
@MattFenelon MattFenelon deleted the Fix-thread-pool-scope branch March 22, 2024 13:31
github-actions bot pushed a commit that referenced this pull request Mar 22, 2024
## [1.6.2](v1.6.1...v1.6.2) (2024-03-22)

### Bug Fixes

* thread pool scope and mutex need to be global across all instances of Scope for it to be a global thread pool ([#471](#471)) ([51fb51c](51fb51c))
Copy link

🎉 This PR is included in version 1.6.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

@jkeen
Copy link
Collaborator

jkeen commented Mar 26, 2024

references this issue: #469

jkeen added a commit to jkeen/graphiti that referenced this pull request Mar 26, 2024
…er handle on what's causing the thread-pools to hang
jkeen added a commit to jkeen/graphiti that referenced this pull request Mar 26, 2024
…er handle on what's causing the thread-pools to hang
jkeen added a commit to jkeen/graphiti that referenced this pull request Mar 26, 2024
…er handle on what's causing the thread-pools to hang
jkeen added a commit that referenced this pull request Mar 26, 2024
…n what's causing thread pool hangs

revert: "thread pool scope and mutex need to be global across all instances of Scope for it to be a global thread pool (#471)"
revert: "add thread pool and concurrency_max_threads configuration option (#470)"

This reverts commit 51fb51c.
This reverts commit 697d761.
jkeen added a commit that referenced this pull request Mar 26, 2024
…n what's causing thread pool hangs. refs #469

revert: "thread pool scope and mutex need to be global across all instances of Scope for it to be a global thread pool (#471)"
revert: "add thread pool and concurrency_max_threads configuration option (#470)"

This reverts commit 51fb51c.
This reverts commit 697d761.
github-actions bot pushed a commit that referenced this pull request Mar 26, 2024
## [1.6.3](v1.6.2...v1.6.3) (2024-03-26)

### Bug Fixes

* Remove thread pool executor logic until we get a better handle on what's causing thread pool hangs. refs [#469](#469) ([7941b6f](7941b6f)), closes [#471](#471) [#470](#470)
jkeen pushed a commit to jkeen/graphiti that referenced this pull request Mar 27, 2024
jkeen pushed a commit to jkeen/graphiti that referenced this pull request Mar 27, 2024
## [1.6.2](graphiti-api/graphiti@v1.6.1...v1.6.2) (2024-03-22)

### Bug Fixes

* thread pool scope and mutex need to be global across all instances of Scope for it to be a global thread pool ([graphiti-api#471](graphiti-api#471)) ([51fb51c](graphiti-api@51fb51c))
jkeen added a commit to jkeen/graphiti that referenced this pull request Mar 27, 2024
…n what's causing thread pool hangs. refs graphiti-api#469

revert: "thread pool scope and mutex need to be global across all instances of Scope for it to be a global thread pool (graphiti-api#471)"
revert: "add thread pool and concurrency_max_threads configuration option (graphiti-api#470)"

This reverts commit 51fb51c.
This reverts commit 697d761.
jkeen pushed a commit to jkeen/graphiti that referenced this pull request Mar 27, 2024
## [1.6.3](graphiti-api/graphiti@v1.6.2...v1.6.3) (2024-03-26)

### Bug Fixes

* Remove thread pool executor logic until we get a better handle on what's causing thread pool hangs. refs [graphiti-api#469](graphiti-api#469) ([7941b6f](graphiti-api@7941b6f)), closes [graphiti-api#471](graphiti-api#471) [graphiti-api#470](graphiti-api#470)
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants