Skip to content

Conversation

@383124397
Copy link

@383124397 383124397 commented Mar 13, 2018

We use httpasyncclient as our core async RPC component, but we found a recovering issue a couple days ago.
it is the async client will be stoped because of I/O reactor shutdown when a service encounter a OOM problem, a few seconds later and after a FGC process, the service has been recovered, but the async client cannot be recovered. so the client cannot accept the request unless the service need to reboot.

the code is:
@Override public void run() { try { final IOEventDispatch ioEventDispatch = new InternalIODispatch(handler); connmgr.execute(ioEventDispatch); } catch (final Exception ex) { log.error("I/O reactor terminated abnormally", ex); } finally { status.set(Status.STOPPED); } }
or by close() method.
so there is no way to set status from STOPPED to ACTIVE even by start().

ok2c and others added 30 commits April 29, 2015 09:00
Contributed by Freeman Fang <ffang at apache.org>

git-svn-id: https://svn.apache.org/repos/asf/httpcomponents/httpasyncclient/branches/4.1.x@1748204 13f79535-47bb-0310-9956-ffa450edef68
…t entity to request

Contributed by Dmitry Potapov <dpotapov at yandex-team.ru>

git-svn-id: https://svn.apache.org/repos/asf/httpcomponents/httpasyncclient/branches/4.1.x@1758103 13f79535-47bb-0310-9956-ffa450edef68
ok2c and others added 17 commits January 28, 2017 10:39
…ly established tunnelled connection leased from the pool when CONNECT request results in 407 over connection that cannot be kept alive.

git-svn-id: https://svn.apache.org/repos/asf/httpcomponents/httpasyncclient/branches/4.1.x@1781738 13f79535-47bb-0310-9956-ffa450edef68
…Provider by default when configured to use system properties

git-svn-id: https://svn.apache.org/repos/asf/httpcomponents/httpasyncclient/branches/4.1.x@1784149 13f79535-47bb-0310-9956-ffa450edef68
@ok2c
Copy link
Member

ok2c commented Mar 13, 2018

@383124397 Exactly what kind of changes are you proposing? Your pull request looks completely messed up. Please resolve all conflicts and resubmit.

@garydgregory
Copy link
Member

@383124397 Please provide your patch on top of the HEAD of the 4.1.x branch. It looks like you are based on code from a 4.1.1-SNAPSHOT. Thank you!

@ok2c
Copy link
Member

ok2c commented Mar 15, 2018

@383124397 could you please either update the change-set or close this pull request?

@383124397
Copy link
Author

I am so sorry that I mark a mistake.

@383124397 383124397 closed this Mar 19, 2018
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.

3 participants