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

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)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants