Hello On 2019-Mar-08, Andres Freund wrote:
> > diff --git a/src/bin/pg_dump/pg_dump.c b/src/bin/pg_dump/pg_dump.c > > index e962ae7e913..1de8da59361 100644 > > --- a/src/bin/pg_dump/pg_dump.c > > +++ b/src/bin/pg_dump/pg_dump.c > > @@ -4359,9 +4359,9 @@ binary_upgrade_set_type_oids_by_rel_oid(Archive *fout, > > "SELECT c.reltype AS crel, t.reltype > > AS trel " > > "FROM pg_catalog.pg_class c " > > "LEFT JOIN pg_catalog.pg_class t ON " > > - " (c.reltoastrelid = t.oid) " > > + " (c.reltoastrelid = t.oid AND > > c.relkind <> '%c') " > > "WHERE c.oid = '%u'::pg_catalog.oid;", > > - pg_rel_oid); > > + RELKIND_PARTITIONED_TABLE, > > pg_rel_oid); > > Hm, I know this code isn't generally well documented, but perhaps we > could add a comment as to why we're excluding partitioned tables? I added a short comment nearby. Hopefully that's sufficient. Let's see what the buildfarm members have to say now. (I wondered about putting the comment between two lines of the string literal, but decided against it ...) Thanks! -- Álvaro Herrera https://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
