Re: pgsql: Rework the pg_statistic_ext catalog - Mailing list pgsql-committers

From Tom Lane
Subject Re: pgsql: Rework the pg_statistic_ext catalog
Date
Msg-id 8833.1560647898@sss.pgh.pa.us
Whole thread Raw
In response to pgsql: Rework the pg_statistic_ext catalog  (Tomas Vondra <tomas.vondra@postgresql.org>)
Responses Re: pgsql: Rework the pg_statistic_ext catalog  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-committers
Tomas Vondra <tomas.vondra@postgresql.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



pgsql-committers by date:

Previous
From: Tomas Vondra
Date:
Subject: pgsql: Fix incorrect CREATE STATISTICS example in docs
Next
From: Tom Lane
Date:
Subject: Re: pgsql: Rework the pg_statistic_ext catalog