Performance improvement on FLAGS-only FETCH

* Compute the intersection set of the uids on a FETCH, so we avoid
  iterating through the non-existant UIDs.
* Dispatch FLAGS query to our specialized method, that fetches all the
  flags documents and return objects that only specify one subset of the
  MessagePart interface, apt to render flags quickly with less queries
  overhead.
* Overwrite the do_FETCH command in the imap Server to use fetch_flags.
* Use deferLater for a better dispatch of tasks in the reactor.

1 files changed, 1 insertions, 10 deletions

diff --git a/src/leap/mail/imap/messages.py b/src/leap/mail/imap/messages.py
index b35b808..22de356 100644
--- a/src/leap/mail/imap/messages.py
+++ b/src/leap/mail/imap/messages.py
@@ -662,17 +662,8 @@ class LeapMessage(fields, MailParser, MBoxParser):
             result = first(flag_docs)
         except Exception as exc:
             # ugh! Something's broken down there!
-            logger.warning("FUCKING ERROR ----- getting for UID:", self._uid)
+            logger.warning("ERROR while getting flags for UID: %s" % self._uid)
             logger.exception(exc)
-            try:
-                flag_docs = self._soledad.get_from_index(
-                    fields.TYPE_MBOX_UID_IDX,
-                    fields.TYPE_FLAGS_VAL, self._mbox, str(self._uid))
-                result = first(flag_docs)
-            except Exception as exc:
-                # ugh! Something's broken down there!
-                logger.warning("FUCKING ERROR, 2nd time -----")
-                logger.exception(exc)
         finally:
             return result