Skip to content

sqlite db task queue#87

Open
amv wants to merge 7 commits intosqlitefrom
feature/db-task-queue
Open

sqlite db task queue#87
amv wants to merge 7 commits intosqlitefrom
feature/db-task-queue

Conversation

@amv
Copy link
Member

@amv amv commented Jun 13, 2015

Should eliminate SQLite db access concurrency within single process.

Also contains small fixes to make lint happy.

This is completely untested code. Can you please test this before making a release and pushing this to production? :) :P

@amv
Copy link
Member Author

amv commented Jun 13, 2015

I remembered that we had actual integration tests, so I ran them! Found one bug, but after fixing it, the tests pass like they do in the sqlite branch.

It would be nice to have you run your "load testing" scripts for this version also if you still have them handy.

@amv
Copy link
Member Author

amv commented Jun 13, 2015

Fixed all sqlite integration tests and made the lock global to the whole process, not just the db object.

load testing would still be nice, but i think this is starting to be merge ready..

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants