I would appreciate opinions from the patch authors on this ordering
change (rationale in previous email). I forgot to CC Sergei and Nikita.
> diff --git a/src/bin/psql/describe.c b/src/bin/psql/describe.c
> index 8dca6d8bb4..9bd0bf8356 100644
> --- a/src/bin/psql/describe.c
> +++ b/src/bin/psql/describe.c
> @@ -6288,7 +6288,11 @@ listOpFamilyOperators(const char *access_method_pattern,
> processSQLNamePattern(pset.db, &buf, family_pattern, have_where, false,
> "nsf.nspname", "of.opfname", NULL, NULL);
>
> - appendPQExpBufferStr(&buf, "ORDER BY 1, 2, o.amopstrategy, 3;");
> + appendPQExpBufferStr(&buf, "ORDER BY 1, 2,\n"
> + " o.amoplefttype = o.amoprighttype DESC,\n"
> + " pg_catalog.format_type(o.amoplefttype, NULL),\n"
> + " pg_catalog.format_type(o.amoprighttype, NULL),\n"
> + " o.amopstrategy;");
>
> res = PSQLexec(buf.data);
> termPQExpBuffer(&buf);
--
Álvaro Herrera https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services