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

restart all threads on eval #947

Merged
merged 1 commit into from
Mar 28, 2023
Merged

restart all threads on eval #947

merged 1 commit into from
Mar 28, 2023

Conversation

ko1
Copy link
Collaborator

@ko1 ko1 commented Mar 28, 2023

When a thread keeps a lock, and REPL runs a code which needs the
lock, other threads should make a progress to release the lock.

fix #877

When a thread keeps a lock, and REPL runs a code which needs the
lock, other threads should make a progress to release the lock.

fix #877
@ko1 ko1 merged commit 9d2a5c0 into master Mar 28, 2023
@ko1 ko1 deleted the restart_all_threads_on_eval branch March 28, 2023 09:28
@st0012
Copy link
Member

st0012 commented Mar 28, 2023

This is amazing! Thank you for the fix 🙏

st0012 added a commit to Shopify/debug that referenced this pull request Mar 29, 2023
This makes sure DAP evaluation request also doesn't hang.

See ruby#947 for the original console implementation.
st0012 added a commit to Shopify/debug that referenced this pull request Mar 30, 2023
This makes sure DAP evaluation request also doesn't hang.

See ruby#947 for the original console implementation.
st0012 added a commit to Shopify/debug that referenced this pull request Mar 31, 2023
This makes sure DAP evaluation request also doesn't hang.

See ruby#947 for the original console implementation.
ko1 pushed a commit that referenced this pull request Apr 1, 2023
This makes sure DAP evaluation request also doesn't hang.

See #947 for the original console implementation.
st0012 added a commit to Shopify/debug that referenced this pull request Apr 3, 2023
This makes sure DAP evaluation request also doesn't hang.

See ruby#947 for the original console implementation.
st0012 added a commit to Shopify/debug that referenced this pull request Apr 3, 2023
This makes sure DAP evaluation request also doesn't hang.

See ruby#947 for the original console implementation.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Debugger freezes when triggering timeout inside a timeout block with Puma server
2 participants