I wrote: > Currently looking through the rest of the patch. I'm wondering > about appendStringLiteral: maybe we should kill that entirely > in favor of using PQescapeStringConn? It's not nearly bright > enough about encoding for instance (and it *will* be used in > client-only encodings).
We could make an appendStringLiteralConn, which would do this correctly for most of the utility programs. However there's a problem for pg_restore: it doesn't necessarily have a PGconn at all. (Consider the case of pg_restore producing text output.) It seems that the alternatives are to export PQescapeStringInternal from libpq, or to duplicate its functionality in appendStringLiteral. Don't much like either, but perhaps the second is less bad. Any opinions? regards, tom lane ---------------------------(end of broadcast)--------------------------- TIP 9: In versions below 8.0, the planner will ignore your desire to choose an index scan if your joining column's datatypes do not match