sqlite3worker icon indicating copy to clipboard operation
sqlite3worker copied to clipboard

Removed busy waiting

Open apsc opened this issue 9 years ago • 2 comments

Unnecessary CPU cycles were being wasted in busy waiting happening inside query_results method. Added synchronisation using threading.Event for select queries to improve the performance overall.

apsc avatar Apr 09 '16 10:04 apsc

Hey. Thanks for the fix. This is a great fix to my double back timer. I was the original author to this library and no longer am with the organization. I believe the branch owner no longer is maintaining this code. I am maintaining this library as a fork here.

https://github.com/dashawn888/sqlite3worker

If you would like to put a merge request on the above I will merge it. If not I will add this myself and comment in the code that this is your fix.

dashawn888 avatar Feb 24 '17 01:02 dashawn888

Hi,

Thanks for getting back to me. I would create a merge request in next couple of days for you to review.

Thanks, Aman

On Fri, Feb 24, 2017 at 6:57 AM, Shawn Lee [email protected] wrote:

Hey. Thanks for the fix. This is a great fix to my double back timer. I was the original author to this library and no longer am with the organization. I believe the branch owner no longer is maintaining this code. I am maintaining this library as a fork here.

https://github.com/dashawn888/sqlite3worker

If you would like to put a merge request on the above I will merge it. If not I will add this myself and comment in the code that this is your fix.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/palantir/sqlite3worker/pull/8#issuecomment-282176320, or mute the thread https://github.com/notifications/unsubscribe-auth/ACkWWlfn2ZHdiZIBAFlvSVgvIRiHNNuVks5rfjHxgaJpZM4IDlsu .

apsc avatar Feb 24 '17 09:02 apsc