On Mon, 8 Jan 2024 at 22:21, Tom Lane <tgl@sss.pgh.pa.us> wrote: > > Richard Guo <guofenglinux@gmail.com> writes: > > On Sun, Jan 7, 2024 at 6:41 AM Tom Lane <tgl@sss.pgh.pa.us> wrote: > >> Thanks for the report! I guess we need something like the attached. > > > +1. > > Pushed, thanks for looking at it.
I have changed the status of the commitfest entry to "Committed" as I noticed the patch has already been committed.
Well, the situation seems a little complex here. At first, this thread was dedicated to discussing the 'Examine-simple-variable-for-Var-in-CTE' patch, which has already been pushed in [1]. Subsequently, I proposed another patch 'Propagate-pathkeys-from-CTEs-up-to-the-outer-query' in [2], which is currently under review and is what the commitfest entry for. Later on, within the same thread, another patch was posted as a fix to the first patch and was subsequently pushed in [3]. I believe this sequence of events might have led to confusion.
What is the usual practice in such situations? I guess I'd better to fork a new thread to discuss my proposed patch which is about the 'Propagate-pathkeys-from-CTEs-up-to-the-outer-query'.