Re: pgsql: Rework the pg_statistic_ext catalog

Tomas Vondra <tomas(dot)vondra(at)postgresql(dot)org> writes:
> Rework the pg_statistic_ext catalog

So ... not one of the buildfarm members that are running TAP tests
likes this. The failures look like

# Running: pg_dump --no-sync --file=/Users/tgl/pgsql/src/bin/pg_dump/tmp_check/tmp_test_cSh8/role.sql --role=regress_dump_test_role --schema=dump_test_second_schema postgres
pg_dump: error: query failed: ERROR: permission denied for table pg_statistic_ext
pg_dump: error: query was: SELECT tableoid, oid, stxname, stxnamespace, (SELECT rolname FROM pg_catalog.pg_roles WHERE oid = stxowner) AS rolname FROM pg_catalog.pg_statistic_ext
not ok 4576 - role: pg_dump runs

# Failed test 'role: pg_dump runs'
# at t/002_pg_dump.pl line 3418.

I think probably what's happening is that pg_dump is still trying to dump
directly from the catalog, when what it needs to do now is dump from the
view, in case it's not running as superuser.

regards, tom lane