moparisthebest/k-9
1
0
Fork 0
mirror of https://github.com/moparisthebest/k-9 synced 2025-02-17 07:30:16 -05:00
Code Issues Releases Wiki Activity

When doing a message search, break it into "first 10" and "more" -

Browse Source 
because a long SQL query takes more time before starting to return
results, K-9 was increasingly slow on large folders.

Doing two searches may take marginally more time but starts getting
results back to the UI much faster.
This commit is contained in:
Jesse Vincent 2010-10-21 20:48:28 +00:00
parent e05784d41d
commit d02ddda19b
1 changed files with 23 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

29
src/com/fsck/k9/mail/store/LocalStore.java
View File

@ -816,13 +816,11 @@ public class LocalStore extends Store implements Serializable
{ {
ArrayList<LocalMessage> messages = new ArrayList<LocalMessage>(); ArrayList<LocalMessage> messages = new ArrayList<LocalMessage>();
Cursor cursor = null; Cursor cursor = null;
int i = 0;
try try
{ {
// pull out messages most recent first, since that's what the default sort is cursor = mDb.rawQuery(queryString + " LIMIT 10", placeHolders);
cursor = mDb.rawQuery(queryString, placeHolders);
int i = 0;
while (cursor.moveToNext()) while (cursor.moveToNext())
{ {
LocalMessage message = new LocalMessage(null, folder); LocalMessage message = new LocalMessage(null, folder);
@ -835,11 +833,26 @@ public class LocalStore extends Store implements Serializable
} }
i++; i++;
} }
if (listener != null) cursor.close();
cursor = mDb.rawQuery(queryString + " LIMIT -1 OFFSET 10", placeHolders);
while (cursor.moveToNext())
{ {
listener.messagesFinished(i); LocalMessage message = new LocalMessage(null, folder);
message.populateFromGetMessageCursor(cursor);
messages.add(message);
if (listener != null)
{
listener.messageFinished(message, i, -1);
}
i++;
} }
} }
catch (Exception e)
{
Log.d(K9.LOG_TAG,"Got an exception "+e);
}
finally finally
{ {
if (cursor != null) if (cursor != null)
@ -847,6 +860,10 @@ public class LocalStore extends Store implements Serializable
cursor.close(); cursor.close();
} }
} }
if (listener != null)
{
listener.messagesFinished(i);
}
return messages.toArray(EMPTY_MESSAGE_ARRAY); return messages.toArray(EMPTY_MESSAGE_ARRAY);