Christopher Kings-Lynne <[EMAIL PROTECTED]> writes:
> If you are referring to my patch, Bruce - that does not fix it.  Mine 
> only addresses psql.

It strikes me that maybe your patch should add parens always, rather
than just in the prettyprint case.

> I don't think that pg_dump uses pg_get_constraintdef().

As of 7.4 it does, looks like:

    if (g_fout->remoteVersion >= 70400)
        appendPQExpBuffer(chkquery, "SELECT conname, "
                        "pg_catalog.pg_get_constraintdef(oid) AS consrc "
                          "FROM pg_catalog.pg_constraint "
                          "WHERE contypid = '%s'::pg_catalog.oid",
                          tinfo->oid);
    else
        appendPQExpBuffer(chkquery, "SELECT conname, 'CHECK (' || consrc || ')' AS 
consrc "
                          "FROM pg_catalog.pg_constraint "
                          "WHERE contypid = '%s'::pg_catalog.oid",
                          tinfo->oid);

                        regards, tom lane

---------------------------(end of broadcast)---------------------------
TIP 3: if posting/reading through Usenet, please send an appropriate
      subscribe-nomail command to [EMAIL PROTECTED] so that your
      message can get through to the mailing list cleanly

Reply via email to