search

Crescent-Labs / iqra-api

GNU General Public License v3.0
21 stars 5 forks source link

add multiprocessing endpoint #3

Closed harunalfat closed 5 years ago

harunalfat commented 5 years ago

Use multiprocess.Pool for faster search.

It shortening request time for query "ان الله و ملائكته يصلون على ال" from 1100ms to 581ms.

Query will be faster when it needs to search exact or partial verse. For special case and word search, it return a similar amount of time.

mmmoussa commented 5 years ago

Thank you very much for your contribution! Last November/December I redesigned the Iqra matching algorithm from scratch and deployed the new version to production, but I never pushed the changes to GitHub. Your addition of using multiprocessing would've been useful for the old implementation and may still slightly speed up the new one as well. If you're able to reapply it to the new Iqra implementation and it provides a performance improvement I'd be happy to merge it in!