Jianlin Shi created UIMA-5812:
---------------------------------
Summary: CPMEngine's getCasProcessors() messes up the ae engines
order
Key: UIMA-5812
URL: https://issues.apache.org/jira/browse/UIMA-5812
Project: UIMA
Issue Type: Bug
Components: UIMA
Affects Versions: 3.0.0SDK, 2.10.2SDK, 2.10.0SDK, 2.10.3SDK
Environment: Ubuntu18.04, Java(TM) SE Runtime Environment (build
1.8.0_151-b12)
Reporter: Jianlin Shi
In the current code:
{quote} 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++) {
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++) {
aList.add(kList.get(i));
}
}
}
{quote}
While analysisEngines and consumers are both Hashtables, iterating through the
keys won't preserve the order.
Suggest to use the following instead:
{quote} ArrayList aList = new ArrayList();
for(Object annotator:this.annotatorDeployList){
List kList = (List) annotator;
if (kList != null) {
for (int i = 0; i < kList.size(); i++) {
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++) {
aList.add(kList.get(i));
}
}
}
{quote}
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
