From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Peter Geoghegan <pg(at)bowt(dot)ie> |
Cc: | pgsql-committers <pgsql-committers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: pgsql: Make heap TID a tiebreaker nbtree index column. |
Date: | 2019-03-24 18:02:24 |
Message-ID: | 28074.1553450544@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
Peter Geoghegan <pg(at)bowt(dot)ie> writes:
> On Sun, Mar 24, 2019 at 10:33 AM Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>> At this point I no longer have any faith in the approach of "suppress
>> DETAIL only where we've actually been burnt". I think we should
>> either go ahead and suppress DETAIL in all four places, or bite the
>> bullet and change the DROP ROLE code as I sketched upthread.
> I would like to suppress the dependency.sql stability issue right
> away. I can also suppress the rowsecurity.out output in the same
> commit. I want to fix the problem on the buildfarm first, unless your
> well-principled approach won't take very long. Which seems unlikely.
I think I can probably get that done today, but if you don't want to
wait, feel free to put in the detail-suppression for now.
>> When I was looking at this on Friday, I thought it wouldn't be that
>> hard to make the results stable, at least up to whatever cutoff we
>> want to set on how many objects to sort. But per previous discussion,
>> maybe we don't need an explicit limit? The stringinfo describing the
>> objects is going to consume a good bit more memory than an ObjectAddress
>> array in any case, so if we're not going to sweat about OOM for the
>> message then I'm not sure we need to be paranoid about the sort memory.
> I agree that it isn't worth worrying about an OOM for the sort here.
If we aren't going to worry about that then I think it probably becomes
a pretty simple change. Will look in a little bit.
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Alexander Korotkov | 2019-03-24 18:17:15 | Re: pgsql: tableam: Add tuple_{insert, delete, update, lock} and use. |
Previous Message | Peter Geoghegan | 2019-03-24 17:44:34 | Re: pgsql: Make heap TID a tiebreaker nbtree index column. |