Skip to content

Progressive timeout dev - #3409

Open
zchuango wants to merge 2 commits into
apache:masterfrom
LinQuickDev:progressive_timeout_dev
Open

Progressive timeout dev#3409
zchuango wants to merge 2 commits into
apache:masterfrom
LinQuickDev:progressive_timeout_dev

Conversation

@zchuango

Copy link
Copy Markdown
Contributor

What problem does this PR solve?

Issue Number: resolve #3133 PR Number: #3163

Problem Summary: when Controller response_will_be_read_progressively() it needs background bthread monitor handler to solve progressive reader idle timeout,when hit the progressive reader idle timeout duration should close current client socket connection.

What is changed and the side effects?

Changed:

  1. add HandleIdleProgressiveReader method will run in and when controller call response_will_be_read_progressively method will trigger HandleIdleProgressiveReader monitor idle progressive reader on bthread_timer.
  2. add progressive idle case on http_c++ example http_server with enable_progressive_timeout arg to trigger ProgressiveAttachment write timeout and http_client with progressive and progressive_read_timeout_ms args test the progressive reader idle timeout case.

Side effects:

Performance effects: NO

Breaking backward compatibility: NO


Check List:

@zchuango

Copy link
Copy Markdown
Contributor Author

@wwbmmm @chenBright This feature PR 3163 hadn't been merged into the community before. Recently, there was a conflict during master code synchronization, so I resubmitted the PR using a new branch. I hope it will be reviewed again, looking forward to this feature being merged into the community.

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.

brpc持续下载,设置每一段超时

1 participant