Skip to content

Conversation

@galibey
Copy link
Contributor

@galibey galibey commented Jul 14, 2022

This change is needed for at most once delivery semantic. The client can send a batch and close the socket without waiting for a response. If the server can't write response it should continue to handle incoming requests that are received and waiting to be served.

Related #2481

@galibey galibey requested a review from sehz July 14, 2022 13:57
@sehz sehz added this to the 0.9.32 milestone Jul 15, 2022
Copy link

@sehz sehz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@galibey
Copy link
Contributor Author

galibey commented Jul 15, 2022

bors r+

bors bot pushed a commit that referenced this pull request Jul 15, 2022
This change is needed for `at most once` delivery semantic. The client can send a batch and close the socket without waiting for a response. If the server can't write response it should continue to handle incoming requests that are received and waiting to be served.

Related #2481

Co-authored-by: Alexander Galibey <48586936+galibey@users.noreply.github.com>
@bors
Copy link

bors bot commented Jul 15, 2022

Build failed:

@galibey
Copy link
Contributor Author

galibey commented Jul 15, 2022

bors retry

bors bot pushed a commit that referenced this pull request Jul 15, 2022
This change is needed for `at most once` delivery semantic. The client can send a batch and close the socket without waiting for a response. If the server can't write response it should continue to handle incoming requests that are received and waiting to be served.

Related #2481

Co-authored-by: Alexander Galibey <48586936+galibey@users.noreply.github.com>
@bors
Copy link

bors bot commented Jul 15, 2022

Build failed:

@galibey
Copy link
Contributor Author

galibey commented Jul 15, 2022

bors retry

bors bot pushed a commit that referenced this pull request Jul 15, 2022
This change is needed for `at most once` delivery semantic. The client can send a batch and close the socket without waiting for a response. If the server can't write response it should continue to handle incoming requests that are received and waiting to be served.

Related #2481

Co-authored-by: Alexander Galibey <48586936+galibey@users.noreply.github.com>
@bors
Copy link

bors bot commented Jul 15, 2022

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title Keep serving incoming requests even if socket closed to write [Merged by Bors] - Keep serving incoming requests even if socket closed to write Jul 15, 2022
@bors bors bot closed this Jul 15, 2022
@galibey galibey deleted the feature/2481-read-incoming-requests-on-closed-socket branch July 15, 2022 09:29
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