[REEF-1186] Remove wrong inject annotation on OperatorTopologyImpl

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 0.14
• Component/s: REEF-IO
• Assignee: Dongjoon Hyun
• Reporter: Dongjoon Hyun
• Created: 04/Feb/16 07:46
• Updated: 04/Feb/16 23:35
• Resolved: 04/Feb/16 23:35

Description

OperatorTopologyImpl is annotated as `@Inject`, but it's not injectable due to the repeated argument types in its constructor. It will cause `ClassHierarchyException`.

-  @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) {

Also, all eight usages of OperatorTopologyImpl uses new like the following.

new OperatorTopologyImpl(this.groupName, this.operName, selfId, driverId, new Sender(netService), version);

Since it seems to be a typo, this issue removes the wrong annotation.

Issue Links

• links to: Link to PR