[
https://issues.apache.org/jira/browse/PROTON-530?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Robbie Gemmell closed PROTON-530.
---------------------------------
Resolution: Won't Fix
Closing, the Messenger API was deprecated in proton-j in the 0.16.0 release
(via PROTON-1361) and removed from proton-j in 0.17.0 (via PROTON-1362).
> MessengerImpl.processActive() does not rethrow IOException
> ----------------------------------------------------------
>
> Key: PROTON-530
> URL: https://issues.apache.org/jira/browse/PROTON-530
> Project: Qpid Proton
> Issue Type: Bug
> Components: proton-j
> Affects Versions: 0.6
> Environment: JDK 7
> Reporter: Olivier Letellier
> Labels: messenger
>
> Test case :
> A connection has been established successfully from Messenger in Proton-j,
> and the client is waiting (possibly with a timeout) on recv().
> Then the peer resets this connection.
> Effect :
> In MessengerImpl.processActive(), a IOException("Connection reset by peer")
> is raised and catched. Until now everything is fine.
> A log is emitted, but the exception is not re-thrown, not registered, and the
> method continues to process other connections.
> From the client point of view :
> - if recv() has a positive timeout, a TimeoutException is raised, but the
> client cannot discriminate it from a timeout when nothing is received on a
> well established session. My client, for example, re-do a recv() , exits
> immediately with a TimeoutException, and then consumes 100% of the CPU of the
> JVM.
> - if recv() is blocking (negative timeout), an infinite loop is happening
> inside waitUntil(), then consuming 100% of CPU.
> I would suggest to throw a new RuntimeException(IOException) after having
> done the log in the catch clause.
> This problem may be linked to PROTON-525 and PROTON-214.
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
