Consider reducing cost of VariableImportAdapter from O(nm) to O(n)

XMLWordPrintable

    • Type: Task
    • Resolution: Won't Do
    • Priority: L3 - Default
    • 2.7.0-alpha1, 2.7.0
    • Affects Version/s: None
    • Component/s: backend
    • None

      during variable import mapping currently all entities are iterated per importAdapter O(n*m)

          for (VariableImportAdapter variableImportAdapter : importAdapterProvider.getPlugins()) {
      pluginVariableList = variableImportAdapter.adaptVariables(pluginVariableList);
    }

      we optimize this by changing the adapter interface to only map one entry and provide a getter for the type it handles. We can then loop once through all the entries, calling the appropriate adapter by type making it O( n ) for each and would be done.

        This is the controller panel for Smart Panels app

              Assignee:
              Unassigned
              Reporter:
              Sebastian Bathke
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Created:
                Updated:
                Resolved: