Cannot execute multiple statements with one call about react-native-quick-sqlite HOT 11 OPEN

Are you getting this using batch operations

from react-native-quick-sqlite.

I haven't tried it but looking at the source code, its not using the pzTail param either.

from react-native-quick-sqlite.

Interesting, but you are better off using the executeBatch API, that one will not only execute all your operations but also wrap them in a transaction.

from react-native-quick-sqlite.

Lol, no. Splitting a code injected SQL migration on runtime with some special comments is error prone (which we're doing at the moment) and PITA especially when they are meant to be run in one go either way.

Using transactions should be left to the API consumer to do.

from react-native-quick-sqlite.

ah, how could I've known? You are always free to create a PR

from react-native-quick-sqlite.

My apologies if the tone of my message was inappropriate.

Usually (or at least based on my experience) database libraries/drivers execute all the provided statements, or documents or errors out in cases when they don't. Its really unexpected behavior to silently execute input only partially.

from react-native-quick-sqlite.

well... I partially implemented this, the problem is that I'm not sure it is really compatible with the current API? How would you pass different params for each row of the string? I would also have to modify the metadata object and the result object to return an array instead of a single object, which would increase the complexity a bit.

In any case, here is a partial implementation, it consumes all the statements but the returned data is not correct at all.

https://github.com/OP-Engineering/op-sqlite/tree/consume-entire-statement

from react-native-quick-sqlite.

The API needs to be split between

1. execute only
2. execute and return data

If that doesn't fit in the architecture, throw error instead of failing silently.

from react-native-quick-sqlite.

alright, as you wish. It's implemented in op-sqlite.

from react-native-quick-sqlite.

Why not just improve this library?

from react-native-quick-sqlite.

This one now belongs to Margelo. I started hacking around on the new one out of curiosity and decided to publish it as a new one as it helps me get customers. Nothing against them, just trying to get something back for my time and effort.

from react-native-quick-sqlite.