[ 
http://jira.nuxeo.org/browse/NXP-1899?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=46966#action_46966
 ] 

Florent Guillaume commented on NXP-1899:
----------------------------------------

Yes, these should be fixed for 5.2.


> DocumentModelImpl methods should not catch client exceptions
> ------------------------------------------------------------
>
>                 Key: NXP-1899
>                 URL: http://jira.nuxeo.org/browse/NXP-1899
>             Project: Nuxeo Enterprise Platform
>          Issue Type: Bug
>          Components: Core
>            Reporter: Olivier Grisel
>            Assignee: Bogdan Stefanescu
>            Priority: Major
>             Fix For: 5.2 M4
>
>   Original Estimate: 2 hours
>  Remaining Estimate: 2 hours
>
> DocumentModelImpl tends to catch a lot of ClientException it without handling 
> then wich results in a hidden aborted transaction that causes the next 
> CoreSession / SearchService calls to crash later and  logs several kB of 
> "Could not find stateful bean xxxx" error logs that are very confusing to 
> debug.
> Offending methods are:
> DocumentModelImpl.getACP
> DocumentModelImpl.setACP
> DocumentModelImpl.getDataModel
> DocumentModelImpl.findAdapter
> Those methods should just not catch ClientException. This implies changing 
> the DocumentModel API to append " throws ClientException" to the prototypes 
> of those methods : this will break API compatibility and thus should be 
> advertised approprietly on the mailing list to inform developers tempted to 
> update.
> NB: Lifecycle related method should also throw ClientExceptions if the cannot 
> find a client instead of logging a error and continue as if the problem had 
> not occured.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://jira.nuxeo.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        
_______________________________________________
ECM-tickets mailing list
ECM-tickets@lists.nuxeo.com
http://lists.nuxeo.com/mailman/listinfo/ecm-tickets

Reply via email to