From: | James Coleman <jtc331(at)gmail(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Robert Haas <robertmhaas(at)gmail(dot)com>, "David G(dot) Johnston" <david(dot)g(dot)johnston(at)gmail(dot)com>, Andrew Dunstan <andrew(at)dunslane(dot)net>, "Bossart, Nathan" <bossartn(at)amazon(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Document atthasmissing default optimization avoids verification table scan |
Date: | 2022-03-26 22:29:08 |
Message-ID: | CAAaqYe-_N8VvUBSfc_QfaPVG-R-4FPFAmD2XKKW28h+5PzBX=w@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Fri, Mar 25, 2022 at 5:00 PM Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>
> Robert Haas <robertmhaas(at)gmail(dot)com> writes:
> > I vote for rejecting both of these patches.
>
> I see what James is on about here, but I agree that these specific changes
> don't help much. What would actually be desirable IMO is a separate
> section somewhere explaining the performance characteristics of ALTER
> TABLE. (We've also kicked around the idea of EXPLAIN for ALTER TABLE,
> but that's a lot more work.) This could coalesce the parenthetical
> remarks that exist in ddl.sgml as well as alter_table.sgml into
> something a bit more unified and perhaps easier to follow. In particular,
> it should start by defining what we mean by "table rewrite" and "table
> scan". I don't recall at the moment whether we define those in multiple
> places or not at all, but as things stand any such discussion would be
> pretty fragmented.
>
> regards, tom lane
I think a unified area discussing pitfalls/performance of ALTER TABLE
seems like a great idea.
That being said: given that "as things stand" that "discussion
[already is] pretty fragmented" is there a place for a simpler
improvement (adding a short explanation of this particular hazard) in
the meantime? I don't mean this specific v4 patch -- just in general
(since the patch can be revised of course).
Thanks,
James Coleman
From | Date | Subject | |
---|---|---|---|
Next Message | Thomas Munro | 2022-03-26 23:12:07 | Re: pgsql: Add 'basebackup_to_shell' contrib module. |
Previous Message | James Coleman | 2022-03-26 22:25:36 | Re: Document atthasmissing default optimization avoids verification table scan |