iOS threading issue with openDatabase?

Question

iOS threading issue with openDatabase?

craig-at-rsg opened this issue 7 years ago · comments

I have intermittent crashes when my app starts up on iOS, and my guess is it's related to calling openDatabase from different places. I need to restart my app multiple times before I hit this. Here's a screenshot - if this doesn't point to the cause, let me know and I'll try creating a test app.

craig-at-rsg · Answer 1 · Wed Sep 06 2017 04:56:01 GMT+0800 (China Standard Time)

openDBs is an NSMutableDictionary, which isn't designed for thread safety...

craig-at-rsg · Answer 2 · Wed Sep 06 2017 09:43:48 GMT+0800 (China Standard Time)

I used a thread safe dictionary (https://gist.github.com/steipete/5928916) in my local build for openDBs, and that fixed the problem I was seeing. I tried to branch this plugin, but I haven't gotten that working yet..

Chris Brody · Answer 3 · Wed Sep 06 2017 09:48:43 GMT+0800 (China Standard Time)

On Tue, Sep 5, 2017 at 9:43 PM, craig-at-rsg ***@***.***> wrote: I used a thread safe dictionary (https://gist.github.com/steipete/5928916) in my local build for openDBs, and that fixed the problem I was seeing. I tried to branch this plugin, but I haven't gotten that working yet..

Thanks @craig-at-rsg for the report, details, and workaround. I will deal with this when I get a chance, been extremely busy with some work for a client.

craig-at-rsg · Answer 4 · Wed Sep 06 2017 21:09:05 GMT+0800 (China Standard Time)

Thanks @brodybits, no rush; my fork is working for me.
https://github.com/litehelpers/Cordova-sqlite-storage/compare/storage-master...craig-at-rsg:2.0.4.1?expand=1

Chris Brody · Answer 5 · Tue Oct 31 2017 05:40:43 GMT+0800 (China Standard Time)

Marking this as a bug, with possible data loss risk. Will integrate the proposed workaround / solution in the near future.

Chris Brody · Answer 6 · Mon Nov 06 2017 12:08:59 GMT+0800 (China Standard Time)

Proposed workaround solution and flag fix for #703 are now integrated with cordova-sqlite-ext, will be integrated with this version in the near future.

Chris Brody · Answer 7 · Wed Nov 08 2017 13:51:17 GMT+0800 (China Standard Time)

Proposed workaround solution is now integrated with this plugin version (cordova-sqlite-storage), will be merged into the other plugin versions in the near future. Thanks again for the contribution.