[UIMA-5812] CPMEngine's getCasProcessors() messes up the ae engines order

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Invalid
• Affects Version/s: 2.10.0SDK, 2.10.2SDK, 3.0.0SDK, 2.10.3SDK
• Component/s: UIMA

Environment

Ubuntu18.04, Java(TM) SE Runtime Environment (build 1.8.0_151-b12)

• Assignee: Unassigned
• Reporter: Jianlin Shi
• Created: 10/Jul/18 03:56
• Updated: 15/Jul/19 18:46
• Resolved: 15/Jul/19 18:46

Description

In the current code:

  ArrayList aList = new ArrayList();
    Iterator keyIt = analysisEngines.keySet().iterator();
    while (keyIt.hasNext()) {
      String keyName = (String) keyIt.next();
      List kList = (List) analysisEngines.get(keyName);
      if (kList != null) {
        for (int i = 0; i < kList.size(); i++)

Unknown macro: {           aList.add(kList.get(i));         }

      }
    }
    keyIt = consumers.keySet().iterator();
    while (keyIt.hasNext()) {
      String keyName = (String) keyIt.next();
      List kList = (List) consumers.get(keyName);
      if (kList != null) {
        for (int i = 0; i < kList.size(); i++)

      }
}

While analysisEngines and consumers are both Hashtables, iterating through the keys won't preserve the order.

Suggest to use the following instead:

        ArrayList aList = new ArrayList();
        for(Object annotator:this.annotatorDeployList){
            List kList = (List) annotator;
            if (kList != null) {
                for (int i = 0; i < kList.size(); i++)

Unknown macro: {                     aList.add(kList.get(i));                 }

            }
        }
        for(Object annotator:this.consumerDeployList){
            List kList = (List) annotator;
            if (kList != null) {
                for (int i = 0; i < kList.size(); i++)

            }
        }