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

Adding an option to use a threadpool for Request/Connection processing. #188

Merged
merged 2 commits into from
Jul 22, 2014

Conversation

NiteshKant
Copy link
Member

If HTTP Request processing or TCP connection handling is blocking or is cpu intensive, there isn't a straightforward way to handoff this processing from the channel's eventloop to a different threadpool.

This change adds this feature. The examples for TCP and HTTP are added to rx-netty-examples under cpuintensive package.

If HTTP Request processing or TCP connection handling is blocking or is cpu intensive, there isn't a straightforward way to handoff this processing from the channel's eventloop to a different threadpool.

 This change adds this feature. The examples for TCP and HTTP are added to rx-netty-examples under cpuintensive package.
@NiteshKant NiteshKant added this to the 0.3.10 milestone Jul 18, 2014
@NiteshKant NiteshKant self-assigned this Jul 18, 2014
@cloudbees-pull-request-builder

RxNetty-pull-requests #118 FAILURE
Looks like there's a problem with this pull request

…lete()

This caused any errors not getting propagated to the connect() subscriber.

Also added some debug for a flaky test.
@cloudbees-pull-request-builder

RxNetty-pull-requests #119 SUCCESS
This pull request looks good

NiteshKant added a commit that referenced this pull request Jul 22, 2014
Adding an option to use a threadpool for Request/Connection processing.
@NiteshKant NiteshKant merged commit 1f012bc into ReactiveX:master Jul 22, 2014
@NiteshKant NiteshKant removed their assignment Aug 19, 2014
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.

2 participants