cxf-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Amichai Rothman (JIRA)" <>
Subject [jira] [Commented] (DOSGI-184) Split Endpoint repository from TopologyManagerExport
Date Mon, 20 May 2013 13:17:17 GMT


Amichai Rothman commented on DOSGI-184:

+1 on splitting it, the topology manager and some other components could sure use some simplification.

The fix on trunk has a little bug though: EndpointRepository.getServices() leaks the inner
structure to its caller(s), which then access it without proper synchronization. The simplest
fix would be to return a copy of the set, although filtering via isAlreadyExportedForRsa internally
and then returning only the non-exported services might also be nice, adding a bit more encapsulation.

Also, the class is public but none of its methods are (they are package-visible) - it would
be clearer if the methods were made public as well.

> Split Endpoint repository from TopologyManagerExport
> ----------------------------------------------------
>                 Key: DOSGI-184
>                 URL:
>             Project: CXF Distributed OSGi
>          Issue Type: Improvement
>    Affects Versions: 1.4.0
>            Reporter: Christian Schneider
>            Assignee: Christian Schneider
>             Fix For: 1.5
> The TopologyManager is a bit difficult to understand as it manages the known services
map and also has to do the communication with ServiceListener, EndpointListenerNotifer and
> Especially the synchronized blocks make the code complicated. 
> So I propose to split the handling of the map of known Endpoints into a EndpointRepository.

This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see:

View raw message