Hi
> Regex matching is obviously unnecessary when we're looking for an exact > match. This checks for this (common) case and starts using plain > equality in that case. +1 > + appendPQExpBuffer(buf, "(%s OPERATOR(pg_catalog.=) ", namevar); > + appendStringLiteralConn(buf, &namebuf.data[2], conn); > + appendPQExpBuffer(buf, "\n OR %s OPERATOR(pg_catalog.=) ", > + altnamevar); > + appendStringLiteralConn(buf, &namebuf.data[2], conn); > + appendPQExpBufferStr(buf, ")\n"); Do we need to force Collaction here like in other branches? if (PQserverVersion(conn) >= 120000) appendPQExpBufferStr(buf, " COLLATE pg_catalog.default");