On Sun, Mar 27, 2022 at 1:00 PM James Coleman <jtc...@gmail.com> wrote: > So "undocumented concept" is just not accurate, and so I don't see it > as a valid reason to reject the patch.
I mean, I think it's pretty accurate. The fact that you can point to a few uses of the terms "table rewrite" and "table scan" in the ALTER TABLE documentation doesn't prove that those terms are defined there or systematically discussed and it seems pretty clear to me that they are not. And I don't even know what we're arguing about here, because elsewhere in the same email you agree that it is reasonable to critique the patch on the basis of how well it fits into the documentation and at least for me that is precisely this issue. I think the bottom line here is that you're not prepared to accept as valid any opinion to the effect that we shouldn't commit these patches. But that remains my opinion. -- Robert Haas EDB: http://www.enterprisedb.com