Feedback-ID: ie0a040ee:Fastmail
Message-ID: <152f375a-8474-4fe5-875b-b78962320515@eisentraut.org>
Date: Thu, 4 Jun 2026 12:04:25 +0200
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: FOR PORTION OF does not recompute GENERATED STORED columns that
 depend on the range column
To: jian he <jian.universality@gmail.com>,
 Paul A Jungwirth <pj@illuminatedcomputing.com>
Cc: Chao Li <li.evan.chao@gmail.com>,
 SATYANARAYANA NARLAPURAM <satyanarlapuram@gmail.com>,
 PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
References: 
 <CAHg+QDcd=t69gLf9yQexO07EJ2mx0Z70NFHo6h94X1EDA=hM0g@mail.gmail.com>
 <CA+renyWk7kVsZJPZKzN95mYkO7S=hDUx=+fUPtbg9qFqeepCpg@mail.gmail.com>
 <CACJufxHai+HB1gkNqVEHe4oKyUmXfAagWBYAWXYKy8hyMV3RxA@mail.gmail.com>
 <CA+renyUBQdhnYxfPay+dxFs6BU1-fnEQskT0r-3dQ2v-ZnmZzg@mail.gmail.com>
 <CA+renyVZLYSghHb_85w0pUBG0KNGKTwciFTKBK5--rpHUM+VdA@mail.gmail.com>
 <e856c439-bf96-4244-8dc3-18b2a552e406@eisentraut.org>
 <CA+renyU6rNkiNGreMyQ7pU_F6-5RND5jchHbECH4NoRO7W0Q-Q@mail.gmail.com>
 <27BD5D23-19C9-4FD1-8935-9C788C3C9869@gmail.com>
 <66C1555B-CA54-4ED1-AB4F-0EE97D24A006@gmail.com>
 <91B35E0F-5DC1-4417-A1B9-FAF4A3DCD2BD@gmail.com>
 <CA+renyUBLAynaj0BKhajB6F=sLuitQkjT+_sOt5HBSRn82iQsw@mail.gmail.com>
 <74C1863C-2C2A-423A-BDE7-0228889F1D80@gmail.com>
 <CA+renyUTzwAMar173cbbxJypChp7s=txxgB+LYJQ5oRZ3a5hYQ@mail.gmail.com>
 <CA+renyXyLfvtvVv--hGWGTgzFP=-+dPLy4RWvEmioAPyJMM+uw@mail.gmail.com>
 <CACJufxH1vFMHg9cT3rS0K6N2nODKSFmgvOtCpHxg7ZdtoQHDQw@mail.gmail.com>
Content-Language: en-US
From: Peter Eisentraut <peter@eisentraut.org>
In-Reply-To: 
 <CACJufxH1vFMHg9cT3rS0K6N2nODKSFmgvOtCpHxg7ZdtoQHDQw@mail.gmail.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit
Archived-At: 
 <https://www.postgresql.org/message-id/152f375a-8474-4fe5-875b-b78962320515%40eisentraut.org>
Precedence: bulk

On 28.05.26 05:23, jian he wrote:
> On Tue, May 26, 2026 at 6:23 AM Paul A Jungwirth
> <pj@illuminatedcomputing.com> wrote:
>>
>> Here is a patch adding the application-time column to updatedCols,
>> along with the tests we've written for GENERATED columns and UPDATE OF
>> triggers.
>>
> 
> V13 looks good to me.
> 
> one minor issue:
> V13 uses the wording "generated stored column(s)", but all other
> places use "stored generated column(s)".

I fixed that.  In one case the test actually included both kinds of 
generated columns, so I just removed the "stored".