Ticket #2619: sql_query_optimization.diff

plugins_base/Logger.py

@@ -233 +233 @@
         '''get the closest timestamp for a nick change, return None
         if no stamp is available'''
 
-        result = self.query('select e.stamp \
-        from event e, user_event ue, user u \
-        where ue.id_user = u.id and \
-        ue.id_event = e.id and \
-        e.stamp < %s and \
-        u.account = "%s" and \
-        e.name = "nick-changed" \
-        order by e.stamp desc limit 1' % (timestamp, contact))
+        query = '''
+        select e.stamp
+        from event as e, (select id_event from user_event where
+        id_user=(select id from user where account='%s')) as ue
+        where e.id=ue.id_event and
+        e.name='nick-changed' and
+        e.stamp < %s
+        order by stamp desc
+        limit 1
+        '''
+        result = self.query(query % (contact, timestamp))
 
         if len(result) == 0:
             return None
@@ -254 +257 @@
         one, when we get to that stamp, we get the new nickname,
         return None if no result'''
 
-        result = self.query('select e.stamp \
-        from event e, user_event ue, user u \
-        where ue.id_user = u.id and \
-        ue.id_event = e.id and \
-        e.stamp >= %s and \
-        u.account = "%s" and \
-        e.name = "nick-changed" limit 1' % (timestamp, contact))
+        query = '''
+        select e.stamp
+        from event as e, (select id_event from user_event where
+        id_user=(select id from user where account='%s')) as ue
+        where e.id=ue.id_event and
+        e.name='nick-changed' and
+        e.stamp >= %s
+        order by stamp desc
+        limit 1
+        '''
+        result = self.query(query % (contact, timestamp))
 
         if len(result) == 0:
             return None
@@ -277 +284 @@
         if stamp is None:
             return contact
 
-        return self.query('select ue.data \
-        from user_event ue, event e \
-        where e.stamp = %f and \
-        ue.id_event = e.id' % (stamp,))[0][0]
+        query = '''
+        select ue.data
+        from user_event ue, (select id from event where stamp = %f) as e
+        where ue.id_event = e.id limit 1
+        '''
+
+        return self.query(query % (stamp,))[0][0]
     
     def get_conversation_ids(self, account):
         '''return the ids of the conversations where the contact was present'''
         
-        query = '''select distinct c.id, c.started 
-        from conversation c, event e, conversation_event ce, user u 
-        where c.id = ce.id_conversation and 
-        e.id = ce.id_event and 
-        ce.id_user = u.id and 
-        u.account = "%s"
+        query = '''
+        select distinct c.id, c.started
+        from conversation c, event e, (select id_conversation,id_event from
+        conversation_event where id_user = (select id from user
+        where account = "%s")) as ce
+        where c.id = ce.id_conversation and
+        e.id = ce.id_event
         order by c.started'''
 
         return self.query(query % (account,))
@@ -300 +311 @@
         account was present, the format of the list is a list of lists
         with the timestamp, email and message'''
 
-        query = '''select e.stamp, u.account, ce.data
-        from event e, conversation_event ce, user u 
-        where e.id = ce.id_event and 
-        u.id = ce.id_user and 
-        ce.id_conversation = %s and
-        e.name = "message" 
+        query = '''
+        select e.stamp, u.account, ce.data
+        from event e, user u, (select id_event,id_user,data
+        from conversation_event where id_conversation = %s) as ce
+        where e.id = ce.id_event and
+        e.name = "message" and
+        u.id = ce.id_user
         limit %s'''
 
         return self.query(query % (id_conversation, num))
@@ -478 +490 @@
         a list of lists with the timestamp and the status'''
 
         query = '''
-        select e.stamp, ue.data 
-        from event e, user_event ue, user u
-        where e.id = ue.id_event and
-        u.id = ue.id_user and
-        e.name = "status-changed" and
-        u.account = "%s"
-        order by e.stamp desc
+        select e.stamp, ue.data
+        from event as e, (select id_event,data from user_event where
+        id_user=(select id from user where account='%s')) as ue
+        where e.id=ue.id_event and
+        e.name='status-changed'
+        order by stamp desc
         limit %s
         '''
 
@@ -496 +507 @@
         a list of lists with the timestamp and the nick'''
 
         query = '''
-        select e.stamp, ue.data 
-        from event e, user_event ue, user u
-        where e.id = ue.id_event and
-        u.id = ue.id_user and
-        e.name = "nick-changed" and
-        u.account = "%s"
-        order by e.stamp desc
+        select e.stamp, ue.data
+        from event as e, (select id_event,data from user_event where
+        id_user=(select id from user where account='%s')) as ue
+        where e.id=ue.id_event and
+        e.name='nick-changed'
+        order by stamp desc
         limit %s
         '''
 
@@ -514 +524 @@
         a list of lists with the timestamp and the pm'''
 
         query = '''
-        select e.stamp, ue.data 
-        from event e, user_event ue, user u
-        where e.id = ue.id_event and
-        u.id = ue.id_user and
-        e.name = "personal-message-changed" and
-        u.account = "%s"
-        order by e.stamp desc
+        select e.stamp, ue.data
+        from event as e, (select id_event,data from user_event where
+        id_user=(select id from user where account='%s')) as ue
+        where e.id=ue.id_event and
+        e.name='personal-message-changed'
+        order by stamp desc
         limit %s
         '''
 
@@ -565 +574 @@
         a list of lists with the timestamp and the message'''
 
         query = '''
-        select e.stamp, ce.data 
-        from event e, conversation_event ce, user u
-        where e.id = ce.id_event and
-        u.id = ce.id_user and
-        e.name = "message" and
-        u.account = "%s"
-        order by e.stamp desc
+        select e.stamp, ce.data
+        from event as e, (select id_event,data from conversation_event where
+        id_user=(select id from user where account='%s')) as ce
+        where e.id=ue.id_event and
+        e.name='message'
+        order by stamp desc
         limit %s
         '''
 
@@ -583 +591 @@
         a list of lists with the timestamp and the dp'''
 
         query = '''
-        select e.stamp, ue.data 
-        from event e, user_event ue, user u
-        where e.id = ue.id_event and
-        u.id = ue.id_user and
-        e.name = "display-picture" and
-        u.account = "%s"
-        order by e.stamp desc
+        select e.stamp, ue.data
+        from event as e, (select id_event,data from user_event where
+        id_user=(select id from user where account='%s')) as ue
+        where e.id=ue.id_event and
+        e.name='display-picture'
+        order by stamp desc
         limit %s
         '''
 
@@ -601 +608 @@
         a list of lists with the timestamp and the ce'''
 
         query = '''
-        select e.stamp, ce.data 
-        from event e, conversation_event ce, user u
-        where e.id = ce.id_event and
-        u.id = ce.id_user and
-        e.name = "custom-emoticon" and
-        u.account = "%s"
-        order by e.stamp desc
+        select e.stamp, ce.data
+        from event as e, (select id_event,data from conversation_event where
+        id_user=(select id from user where account='%s')) as ce
+        where e.id=ue.id_event and
+        e.name='custom-emoticon'
+        order by stamp desc
         limit %s
         '''
 
@@ -618 +624 @@
         account was present, the format of the list is a list od lists
         with the timestamp, email and message'''
 
-        # in the inner select we select all the id of conversations
-        # where account was present, then on the outer conversation
-        # we select all the messages from that conversations
         query = '''
         select e.stamp, u.account, ce.data
-        from event e, conversation_event ce, user u 
-        where e.id = ce.id_event and 
-        u.id = ce.id_user and 
-        ce.id_conversation in 
-        (select distinct c.id 
-        from conversation c, event e, conversation_event ce, user u 
-        where c.id = ce.id_conversation and 
-        e.id = ce.id_event and 
-        ce.id_user = u.id and 
-        u.account = "%s") and 
-        e.name = "message" 
+        from event e, user u,
+        (select id_event,id_user,id_conversation,data from
+        conversation_event where id_user = (select id from
+        user where account = "%s")) as ce
+        where e.name = "message" and
+        e.id = ce.id_event and
+        u.id = ce.id_user
         order by e.stamp desc
         limit %s
         '''