[UIMA-1365] Code trying to ensure reverse iterators list items in the opposite order of forward ones is present but not working

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 2.3
• Component/s: Core Java Framework
• Assignee: Unassigned
• Reporter: B Lambov (old handle)
• Created: 02/Jun/09 13:43
• Updated: 15/Sep/09 15:23
• Resolved: 03/Jun/09 15:17

Description

FSIndexRepositoryImpl contains pieces of code that are meant to ensure forward and reverse iterators are inverses of each other by doing additional comparison by FS address to resolve equality. However, this is not fully implemented and isn't working properly.

The iteratorComparator defined by the PointerIterator class is not used for sorting elements in moveToNext() and moveToPrevious(), where the default comparators for the iterators are used instead. For the typical use of PointerIterator of listing the annotation index this does not cause a problem as the type priorities ensure ordering between indexes of different types, while the ordering is always defined within the individual indexes.

The piece of code implementing the extra comparison should either be removed or corrected.

Issue Links

• is part of: UIMA-1366 Binary heap annotation iterator implementation
• is related to: UIMA-190 Type priority test case failing with IBM JDK 1.5