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
