Re: HOT readme missing documentation on summarizing index handling

2023-07-10 Thread Aleksander Alekseev
Hi, > > Thanks, pushed after correcting a couple typos. > > Thanks! I noticed that ec99d6e9c87a introduced a slight typo: s/if there is not room/if there is no room -- Best regards, Aleksander Alekseev

Re: HOT readme missing documentation on summarizing index handling

2023-07-07 Thread Matthias van de Meent
On Fri, 7 Jul 2023 at 19:06, Tomas Vondra wrote: > > On 7/7/23 18:34, Matthias van de Meent wrote: > > On Fri, 7 Jul 2023 at 00:14, Tomas Vondra > > wrote: > >> The original text was really about on/off, and I'm not quite sure the > >> part about "exception" makes this very clear. > > > > Agreed

Re: HOT readme missing documentation on summarizing index handling

2023-07-07 Thread Tomas Vondra
On 7/7/23 18:34, Matthias van de Meent wrote: > On Fri, 7 Jul 2023 at 00:14, Tomas Vondra > wrote: >> >> Yeah, README.HOT should have been updated, and I see no reason not to >> backpatch this to v16. Barring objections, I'll do that tomorrow. >> >> I have two suggesting regarding the README.HOT

Re: HOT readme missing documentation on summarizing index handling

2023-07-07 Thread Matthias van de Meent
On Fri, 7 Jul 2023 at 00:14, Tomas Vondra wrote: > > Yeah, README.HOT should have been updated, and I see no reason not to > backpatch this to v16. Barring objections, I'll do that tomorrow. > > I have two suggesting regarding the README.HOT changes: > > 1) I'm not entirely sure it's very clear wh

Re: HOT readme missing documentation on summarizing index handling

2023-07-06 Thread Tomas Vondra
Yeah, README.HOT should have been updated, and I see no reason not to backpatch this to v16. Barring objections, I'll do that tomorrow. I have two suggesting regarding the README.HOT changes: 1) I'm not entirely sure it's very clear what "referential integrity of indexes across tuple updates" act

HOT readme missing documentation on summarizing index handling

2023-07-06 Thread Matthias van de Meent
Hi, With PG16's 19d8e230, we got rid of BRIN's blocking of HOT updates, but I just realized that we failed to update the README.HOT document with this new exception for summarizing indexes. Attached a patch that updates that document, detailing the related rationale. I'm not sure if such interna