Skip to content

gh-84570: Implement Waiting in SendChannel.send() #110565

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

Merged

Conversation

ericsnowcurrently
Copy link
Member

@ericsnowcurrently ericsnowcurrently commented Oct 9, 2023

We had been faking it (poorly).

We will add timeouts separately.

@ericsnowcurrently ericsnowcurrently changed the title Implement Waiting in SendChannel.send() gh-84570: Implement Waiting in SendChannel.send() Oct 9, 2023
@ericsnowcurrently ericsnowcurrently enabled auto-merge (squash) October 10, 2023 09:23
@ericsnowcurrently ericsnowcurrently merged commit 757cc35 into python:main Oct 10, 2023
@ericsnowcurrently ericsnowcurrently deleted the channel-send-wait-new branch October 10, 2023 09:38
ericsnowcurrently added a commit that referenced this pull request Oct 17, 2023
There were a few things I did in gh-110565 that need to be fixed. I also forgot to add tests in that PR.

(Note that this PR exposes a refleak introduced by gh-110246. I'll take care of that separately.)
aisk pushed a commit to aisk/cpython that referenced this pull request Feb 11, 2024
There were a few things I did in pythongh-110565 that need to be fixed. I also forgot to add tests in that PR.

(Note that this PR exposes a refleak introduced by pythongh-110246. I'll take care of that separately.)
Glyphack pushed a commit to Glyphack/cpython that referenced this pull request Sep 2, 2024
)

We had been faking it (poorly).

We will add timeouts separately.
Glyphack pushed a commit to Glyphack/cpython that referenced this pull request Sep 2, 2024
There were a few things I did in pythongh-110565 that need to be fixed. I also forgot to add tests in that PR.

(Note that this PR exposes a refleak introduced by pythongh-110246. I'll take care of that separately.)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant