From: | Justin Pryzby <pryzby(at)telsasoft(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | David Gilman <davidgilman1(at)gmail(dot)com>, pgsql-hackers(at)lists(dot)postgresql(dot)org |
Subject: | Re: "tuple concurrently updated" in pg_restore --jobs |
Date: | 2020-07-10 21:45:21 |
Message-ID: | 20200710214521.GS26220@telsasoft.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Fri, Jul 10, 2020 at 05:36:28PM -0400, Tom Lane wrote:
> Justin Pryzby <pryzby(at)telsasoft(dot)com> writes:
> > On Fri, Jul 10, 2020 at 04:54:40PM -0400, Tom Lane wrote:
> >> This works about 99% of the time, in fact. It falls down in the --clean
>
> > Note that this fails for me (sometimes) even without --clean.
>
> Oh, I was thinking that REVOKE would only be issued in the --clean
> case, but apparently that's not so. Doesn't really affect the fix
> proposal though. I just finished a patch for HEAD, as attached.
>
> (I flushed the "CatalogId objCatId" argument of dumpACL, which was
> not used.)
>
> I'm not sure how far to back-patch it -- I think the parallel restore
> of ACLs behavior is not very old, but we might want to teach older
> pg_dump versions to insert the extra dependency anyway, for safety.
Yes, and the test case in David's patch on other thread [0] can't be
backpatched further than this patch is. A variant on his test case could just
as well be included in this patch (with pg_dump writing to a seekable FD) and
then amended later to also test writing to an unseekable FD.
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2020-07-10 21:47:38 | Re: Stale external URL in doc? |
Previous Message | Thomas Munro | 2020-07-10 21:42:17 | Re: Stale external URL in doc? |