Uploaded image for project: 'OpenFlowPlugin'
  1. OpenFlowPlugin
  2. OPNFLWPLUG-656

Translate/convert in Services (remove dependency on DeviceContext)

    XMLWordPrintable

Details

    • Bug
    • Status: Resolved
    • Resolution: Done
    • None
    • None
    • General
    • None
    • Operating System: All
      Platform: All

    • 5591

    Description

      Call DeviceContext.oook() before creating instance of the service (encapsulate to util method). For classes that are currently calling DeviceContext.oook() add parameter to the constructor:
      a/ org.opendaylight.openflowplugin.impl.statistics.services.AggregateFlowsInTableService
      b/ org.opendaylight.openflowplugin.impl.statistics.services.OpendaylightFlowStatisticsServiceImpl

      Call getMessageConverter(key) before creating instance of the service (encapsulate to util method). Add argument ConvertorMessageToOFJava to constructor of class and stop using getDeviceContext():
      c/ org.opendaylight.openflowplugin.impl.services.SalExperimenterMessageServiceImpl

      org.opendaylight.openflowplugin.api.openflow.device.MessageTranslator<I, O>
      a/ org.opendaylight.openflowplugin.impl.translator.AggregatedFlowStatisticsTranslator
      b/ org.opendaylight.openflowplugin.impl.translator.FlowRemovedTranslator
      c/ org.opendaylight.openflowplugin.impl.translator.FlowRemovedV10Translator
      d/ org.opendaylight.openflowplugin.impl.translator.PacketReceivedTranslator (see OPNFLWPLUG-655 PortNumberCache)
      e/ org.opendaylight.openflowplugin.impl.translator.PortUpdateTranslator

      Attachments

        Issue Links

          No reviews matched the request. Check your Options in the drop-down menu of this sections header.

          Activity

            People

              tomas.slusny@pantheon.tech Tomas Slusny
              jozef.slezak@pantheon.sk Jozef Slezák
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: