Database attach flow fixed.

[dryganets]

Threading model fixed in order to have all queries executed in the same order. android-native plugin fixed.

@itinance please check this change locally. Your initial change doesn't even compile in android-native. Also it has hard to figure out race condition. So if you have this code running in production I have a bad news for you.

Though I didn't test this as well.
We don't rely on this feature, I just see that it's implemented the way it could cause weird crashes.

[itinance]

Wow! Thank you very much! Will test in upcoming days. One of our applications is using ATTACH DATABASE very heavily.
But we never used android-native plugin. Maybe this is the reason that we never noticed issues with that. How did you activate it? What do i have to do to active the native-plugin against the other one.
I was quiet lost those days with the android-part (i am iOS-native-developer and this was my first JAVA-contribution). You can read all this in all these comments on my pull request then ;), also my question why there are two different implementations for android because they are looking quiet the same ^^

[dryganets]

@itinance, one implementation is using system SQLite (android).
And another one uses separately built distribution.
I think it's there historically probably someone experienced some weird issues with system library on Cordova. And it also easier to support your application if you have the same version of SQLite among all android devices and API levels.

to test it just replace android by android-native in settings.gradle of your project.

[andpor]

@itinance can you double check these changes please?

[itinance]

Confirmed.