[jira] [Resolved] (REEF-1186) Remove wrong inject annotation on OperatorTopologyImpl

["Markus Weimer (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/REEF-1186?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Markus Weimer resolved REEF-1186.
---------------------------------
       Resolution: Fixed
    Fix Version/s: 0.14

Resolved via [#821|https://github.com/apache/reef/pull/821]

> Remove wrong inject annotation on OperatorTopologyImpl
> ------------------------------------------------------
>
>                 Key: REEF-1186
>                 URL: https://issues.apache.org/jira/browse/REEF-1186
>             Project: REEF
>          Issue Type: Bug
>          Components: REEF-IO
>            Reporter: Dongjoon Hyun
>            Assignee: Dongjoon Hyun
>            Priority: Minor
>             Fix For: 0.14
>
>
> *OperatorTopologyImpl* is annotated as `@Inject`, but it's not injectable due to the repeated argument types in its constructor. It will cause `ClassHierarchyException`. 
> {code}
> -  @Inject
>    public OperatorTopologyImpl(
>       final Class<? extends Name<String>> groupName,
>       final Class<? extends Name<String>> operName, final String selfId,
>       final String driverId, final Sender sender, final int version) {
> {code}
> Also, all eight usages of *OperatorTopologyImpl* uses *new* like the following.
> {code}
> new OperatorTopologyImpl(this.groupName, this.operName, selfId, driverId, new Sender(netService), version);
> {code}
> Since it seems to be a typo, this issue removes the wrong annotation.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)