Skip to content
This repository has been archived by the owner on Oct 2, 2019. It is now read-only.

deleteMessage() and markMessageRead() should be able to accept an array of IDs #81

Open
airpingu opened this issue Aug 29, 2013 · 1 comment

Comments

@airpingu
Copy link

Someone used to mention we've already fired a bug for this issue but I cannot find that so I open a new one. Please feel free to close it if duplicated.

In our current Mozilla implementation, we found eating an array of IDs can be very useful to improve the performance. Our experiments show it has to take around 30~40 seconds to delete 2000 messages if the delete method can only accept single ID. However, it can take just 5 seconds to delete all of them if the delete method can accept multiple IDs, because the back-end can simply open one DB transaction to finish the task from the point of view of implementation.

We might need to do the same thing for deleteConversation() and markConversationRead().

@zolkis
Copy link
Contributor

zolkis commented Oct 29, 2013

+1
BTW, a previous version contained arrays, but someone complained in the first F2F, so we have removed it :).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants