Re: Ignoring BRIN for HOT udpates seems broken - Mailing list pgsql-hackers

From Robert Haas
Subject Re: Ignoring BRIN for HOT udpates seems broken
Date
Msg-id CA+TgmoZ8skvW3RNQV_ok0yzynHBQSwpJ9YbETySq-08zxLJBbg@mail.gmail.com
Whole thread Raw
In response to Re: Ignoring BRIN for HOT udpates seems broken  (Tomas Vondra <tomas.vondra@enterprisedb.com>)
Responses Re: Ignoring BRIN for HOT udpates seems broken
List pgsql-hackers
On Sat, May 28, 2022 at 4:51 PM Tomas Vondra
<tomas.vondra@enterprisedb.com> wrote:
> Yeah, I think that might/should work. We could still create the HOT
> chain, but we'd have to update the BRIN indexes. But that seems like a
> fairly complicated change to be done this late for PG15.

Yeah, I think a revert is better for now. But I agree that the basic
idea seems salvageable. I think that the commit message is correct
when it states that "When determining whether an index update may be
skipped by using HOT, we can ignore attributes indexed only by BRIN
indexes." However, that doesn't mean that we can ignore the need to
update those indexes. In that regard, the commit message makes it
sound like all is well, because it states that "the page range summary
will be updated anyway" which reads to me like the indexes are in fact
getting updated. Your example, however, seems to show that the indexes
are not getting updated.

-- 
Robert Haas
EDB: http://www.enterprisedb.com



pgsql-hackers by date:

Previous
From: Zhihong Yu
Date:
Subject: Re: Add SPLIT PARTITION/MERGE PARTITIONS commands
Next
From: Andres Freund
Date:
Subject: Re: [RFC] building postgres with meson -v8