I wrote: > Yes, that's the expectation. I'm sure we can think of a more > backwards-compatible way to test for empty datacl, though.
Looks like the attached should be sufficient. As penance, I tried all the commands in describe.c against 9.2 (or the oldest supported server version), and none of them fail now. regards, tom lane
diff --git a/src/bin/psql/describe.c b/src/bin/psql/describe.c index faabecbc76..6a36c91083 100644 --- a/src/bin/psql/describe.c +++ b/src/bin/psql/describe.c @@ -6678,7 +6678,7 @@ printACLColumn(PQExpBuffer buf, const char *colname) { appendPQExpBuffer(buf, "CASE" - " WHEN pg_catalog.cardinality(%s) = 0 THEN '%s'" + " WHEN pg_catalog.array_length(%s, 1) = 0 THEN '%s'" " ELSE pg_catalog.array_to_string(%s, E'\\n')" " END AS \"%s\"", colname, gettext_noop("(none)"),