Re: DOCS: add helpful partitioning links
От | Robert Treat |
---|---|
Тема | Re: DOCS: add helpful partitioning links |
Дата | |
Msg-id | CAJSLCQ0Eiv5-Dhe7TZd4biA1vG1cua01ZHkUKSmccMewdufgHg@mail.gmail.com обсуждение исходный текст |
Ответ на | Re: DOCS: add helpful partitioning links (Ashutosh Bapat <ashutosh.bapat.oss@gmail.com>) |
Ответы |
Re: DOCS: add helpful partitioning links
|
Список | pgsql-hackers |
On Thu, Mar 21, 2024 at 7:27 AM Ashutosh Bapat <ashutosh.bapat.oss@gmail.com> wrote: > On Wed, Mar 20, 2024 at 5:22 PM Ashutosh Bapat <ashutosh.bapat.oss@gmail.com> wrote: >> On Tue, Mar 19, 2024 at 6:38 PM Robert Treat <rob@xzilla.net> wrote: >>> >>> >>> I've put it in the next commitfest with target version of 17, and I've >>> added you as a reviewer :-) >>> >> >> Thanks. >> >>> >>> Also, attached is an updated patch with your change above which should >>> apply cleanly to the current git master. >> >> >> It did apply for me now. >> >> The HTML renders good, the links work as expected. >> >> The CREATE TABLE ... LIKE command >> I think the original word "option" instead of "command" is better since we are talking about LIKE as an option to CREATETABLE instead of CREATE TABLE command. >> >> + but any future attached or created partitions will be indexed as well. >> >> I think just "any future partitions will be indexed as well" would suffice, no need to mention whether they were createdor attached. >> >> + One limitation when creating new indexes on partitioned tables is that it >> + is not possible to use the <literal>CONCURRENTLY</literal> >> + qualifier when creating such a partitioned index. To avoid long >> >> The sentence uses two different phrases, "indexes on partitioned tables" and "partitioned index", for the same thing inthe same sentence. Probably it is better to leave original sentence as is. >> >> But I think it's time for a committer to take a look at this. Please feel free to address the above comments if you agreewith them. Marking this as ready for committer. >> > > The patch changes things not directly related to $Subject. It will be good to add a commit message to the patch describingwhat are those changes about. I observe that all of them are in section "partition maintenance". https://www.postgresql.org/docs/16/ddl-partitioning.html#DDL-PARTITIONING-DECLARATIVE-MAINTENANCE.Do you see any more editsrequired in that section? > Heh, well, I had thought about some other possible improvements to that section but hadn't quite worked them out, but you inspired me to have another go of it ;-) v5 patch attached which I think further improves clarity/brevity of this section. I've left the patch name the same for simplicity, but I'd agree that the commit would now be more along the lines of editing / improvements / copyrighting of "Partition Maintenance" docs. Robert Treat https://xzilla.net
Вложения
В списке pgsql-hackers по дате отправления: