[
https://issues.apache.org/jira/browse/CXF-2398?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12746125#action_12746125
]
William Tam edited comment on CXF-2398 at 8/21/09 11:22 AM:
------------------------------------------------------------
My point regarding the cleanup() is either way (destroy client or not). we are
not 100% compliant with the spec because we either leave the handle as
"validate" or we close the connection. I agree with you that destroying the
client in cleanup is not efficient (I have retrieved my opinion to invoke
cleanup before your comment). However, the current cleanup method needs update
if we are not destroying the connections if it intents to preserve the clients.
(i.e. it should not clear the handles map).
was (Author: wtam):
My point regarding the cleanup() is either way (destroy client or not). we
are not 100% compliant with the spec because we either leave the handle as
"validate" or we close the connection. I agree with you that destroying the
client in cleanup is efficient (I have retrieved my opinion to invoke cleanup
before your comment). However, the current cleanup method needs update if we
are not destroying the connections if it intents to preserve the clients.
(i.e. it should not clear the handles map).
> JMS Connections are not closed when JCA recycles CXF proxy
> ----------------------------------------------------------
>
> Key: CXF-2398
> URL: https://issues.apache.org/jira/browse/CXF-2398
> Project: CXF
> Issue Type: Bug
> Components: Integration
> Affects Versions: 2.1
> Reporter: Seumas Soltysik
> Assignee: Daniel Kulp
> Fix For: 2.1.7, 2.2.4
>
> Attachments: CXF-2398.patch, CXF-2398.patch2
>
>
> When close() is called on JCA ConnectionHandler, there is no cleanup of the
> underlying client proxy and its transport. Specfically when the JCA outbound
> use case is using JMS, JMS connections are not being closed.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
