[MAILBOX-191] more effective implementation of StoreMessageResultIterator

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 0.4, 0.5
• Fix Version/s: 0.5
• Component/s: store
• Labels: patch
• Assignee: Eric Charles
• Reporter: Peter Kvokacka
• Created: 18/Jan/13 15:18
• Updated: 06/Feb/13 06:46
• Resolved: 06/Feb/13 06:46

Description

I found problem with current implementation of StoreMessageResultIterator. I discribed it here.
http://www.mail-archive.com/[e-mail]/msg14242.html
Briefly:
From Thunderbird I got request

452 UID fetch 16,17:18,20:41,43:45,47,49 (UID RFC822.SIZE BODY.PEEK[])

which is mapped to calls on MessageMapper.findInMailbox()

findInMailbox: TYPE: RANGE UID: 16:18, Full
*findInMailbox: TYPE: FROM UID: 19:-1, Full
findInMailbox: TYPE: RANGE UID: 20:41, Full
*findInMailbox: TYPE: FROM UID: 42:-1, Full
findInMailbox: TYPE: RANGE UID: 43:45, Full
*findInMailbox: TYPE: FROM UID: 46:-1, Full
findInMailbox: TYPE: ONE UID: 47:47, Full
*findInMailbox: TYPE: ONE UID: 48:48, Full
findInMailbox: TYPE: ONE UID: 49:49, Full

Those marked calls of method findInMailbox are completely unnecessary and should be avoided.