tursodatabase / libsql-client-ts

TypeScript/JavaScript client API for libSQL
https://docs.turso.tech/sdk/ts/quickstart
MIT License
226 stars 32 forks source link

Remove database probe from `createClient()` #154

Open penberg opened 8 months ago

penberg commented 8 months ago

We probe if database can be opened with SQL in createClient() implementations, but it means that with remote databases, we add extra latency. This is problematic in particular if client is short-lived.

invisal commented 8 months ago

I am new here. But I believe we need to remove this one line https://github.com/tursodatabase/libsql-client-ts/blob/db1daf4e71a63977e15a49fdcdb23f01e6eb4f27/packages/libsql-client/src/sqlite3.ts#L58C34-L58C65

If it is true, let assign it to me.