ctakes-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Tim Miller (JIRA)" <j...@apache.org>
Subject [jira] [Created] (CTAKES-265) isDuplicate iterates over set
Date Tue, 12 Nov 2013 22:37:18 GMT
Tim Miller created CTAKES-265:

             Summary: isDuplicate iterates over set
                 Key: CTAKES-265
                 URL: https://issues.apache.org/jira/browse/CTAKES-265
             Project: cTAKES
          Issue Type: Improvement
          Components: ctakes-dictionary-lookup
            Reporter: Tim Miller
            Priority: Minor

The private method isDuplicate() in DictionaryLookupAnnotator is used to filter out duplicates
from its lookups. It does so by keeping a Set object full of objects its seen before, and
then does a lookup by manually iterating over all the elements. I don't see any reason why
it shouldn't just call the contains() method of Set, which I do not believe could possibly
be slower. Probably much faster in fact. Any thoughts or suggestions?

This message was sent by Atlassian JIRA

View raw message