Uploaded image for project: 'REEF (Retired)'
  1. REEF (Retired)
  2. REEF-1186

Remove wrong inject annotation on OperatorTopologyImpl

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Minor
    • Resolution: Fixed
    • None
    • 0.14
    • REEF-IO
    • None

    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.

      Attachments

        Issue Links

          Activity

            People

              dongjoon Dongjoon Hyun
              dongjoon Dongjoon Hyun
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: