Sehrope Sarkuni
Sehrope Sarkuni
Haven't looked through the PR itself to see if it makes sense, but I can see a valid use for something like this when you're not actually sure what SQL...
The `.native` approach uses libpq under the covers which defaults to an SSL mode of "prefer" which will use SSL: https://www.postgresql.org/docs/current/libpq-ssl.html#LIBPQ-SSL-PROTECTION With the pure-JS driver (i.e. regular pg), you need...
I've put together a WIP patch that corrects this behavior. It adds an `extra_float_digits` connection parameter defaulted to 3. Branch for it is here: https://github.com/sehrope/node-postgres/tree/extra-float-digits Couple of points/issues as it's...
The CRL handling itself is not too complicated as once the list is instantiated, it just gets added as an additional trust manager: https://github.com/pgjdbc/pgjdbc/pull/1297/files#diff-7e6aeeea6d1972e2d336db17384fb6b1e8e308c06dbe3adda554b0ac5c3b4ab2 The real work is going to...
I can't see the driver ever supporting this type of thing, particularly when the SQL would conflict with native PostgreSQL syntax. The only driver I can that supports this syntax...
> Regarding @sehrope 's comment I need to say that it is actually pretty easy to find out if the caller wants native syntax or if he wants escape sequences:...
-1 to doing anything more than we do now (i.e. just closing the socket). If we have to issue cancels then we'd have to wait for them as well. If...
Ah good catch, I forgot it takes an Executor to offload the work to a different thread.
+1 At the very least we should have something in our docs that mentions this error and the likely cause being due to connection pooling / deallocate. Even if we...
You should rename this PR to something like, "Implement PgConnection.createStruct(...) to support custom types". The source branch name in your fork doesn't really matter, but best to use something similar...