[jboss-jira] [JBoss JIRA] (WFLY-4359) WSEndpointDeploymentUnit should implement a WS-specific interface

[Brian Stansberry (JIRA)]

     [ https://issues.jboss.org/browse/WFLY-4359?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Brian Stansberry updated WFLY-4359:
-----------------------------------
    Description: 
WSEndpointDeploymentUnit implements the DeploymentUnit interface, but not really. A bunch of methods throw UnsupportedOperationException.

This tells me it's using that interface as a minor short-cut, not because it's truly meant to be a DeploymentUnit impl.

This is the only impl of this interface outside of WFCORE, and having it here makes it harder to evolve the interface.

The WS subsystem code should use its own interface (e.g. as a param to WSDeploymentBuilder.build()), and for the call paths that want to use the DeploymentUnit from WFCORE, wrap the DU in an impl of this WS interface and delegate to it.

  was:
WSEndpointDeploymentUnit implements the DeploymentUnit interface, but not really. A bunch of methods throw UnsupportedOperationException.

This tells me it's using that interface as a minor short-cut, not because it's truly meant to be a DeploymentUnit impl.

This is the only impl of this interface outside of WFCORE, and having it here makes it harder to evolve the interface.

The WS subsystem code should use its own interface (e.g. as a param to WSDeploymentBuilder.build(), and for the call paths that want to use the DeploymentUnit from WFCORE, wrap the DU in an impl of this WS interface and delegate to it.



> WSEndpointDeploymentUnit should implement a WS-specific interface
> -----------------------------------------------------------------
>
>                 Key: WFLY-4359
>                 URL: https://issues.jboss.org/browse/WFLY-4359
>             Project: WildFly
>          Issue Type: Bug
>          Components: Web Services
>            Reporter: Brian Stansberry
>            Assignee: Alessio Soldano
>
> WSEndpointDeploymentUnit implements the DeploymentUnit interface, but not really. A bunch of methods throw UnsupportedOperationException.
> This tells me it's using that interface as a minor short-cut, not because it's truly meant to be a DeploymentUnit impl.
> This is the only impl of this interface outside of WFCORE, and having it here makes it harder to evolve the interface.
> The WS subsystem code should use its own interface (e.g. as a param to WSDeploymentBuilder.build()), and for the call paths that want to use the DeploymentUnit from WFCORE, wrap the DU in an impl of this WS interface and delegate to it.



--
This message was sent by Atlassian JIRA
(v6.3.11#6341)