Re: Possible performance regression with pg_dump of a large numberof relations - Mailing list pgsql-hackers

From Stephen Frost
Subject Re: Possible performance regression with pg_dump of a large numberof relations
Date
Msg-id 20180124225641.GC2416@tamriel.snowman.net
Whole thread Raw
In response to Re: Possible performance regression with pg_dump of a large number ofrelations  (Luke Cowell <lcowell@gmail.com>)
Responses Re: Possible performance regression with pg_dump of a large number ofrelations
List pgsql-hackers
Hi there!

* Luke Cowell (lcowell@gmail.com) wrote:
> Hi Stephen, thank you for putting this together.

Yeah, it needs more work, which I figured out after actually hacking
together a patch for it and I've just not gotten back to it yet.

> > If folks get a chance to take a look at the query and/or test, that'd be
> > great.  I'll try to work up an actual patch to pg_dump this weekend to
> > run it through the regression tests and see if anything breaks.
>
> I'm not sure how I can help other than testing that this runs. I can confirm that it runs on 10.1. It does not run on
9.5or 9.6 and gives this error: 

Thanks for offering to help!  Once I've got an actual patch together
that works well enough to get through the regression tests, I'll
definitely let you know.  The general premise still looks viable, but
the actual query needs to be reworked.

> > ERROR:  relation "pg_init_privs" does not exist
> > LINE 139: LEFT JOIN pg_init_privs pip

I certainly hope that works on 9.6, since that's when pg_init_privs was
added..

> I'm guessing that the error is not surprising and that the query is intended for an upcoming release of postgres and
wouldn'tbe backported to 9.6.x 

Presuming I can make it work, the idea would be to back-port it to 9.6
and 10, since pg_init_privs and this code was added in 9.6.

Thanks again!

Stephen

Attachment

pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: Converting plpgsql to use DTYPE_REC for named composite types
Next
From: Alvaro Herrera
Date:
Subject: reducing isolation tests runtime