Re: Add RESPECT/IGNORE NULLS and FROM FIRST/LAST options

Paul Ramsey <pramsey@cleverelephant.ca>

From: Paul Ramsey <pramsey@cleverelephant.ca>
To: Tatsuo Ishii <ishii@postgresql.org>
Cc: tgl@sss.pgh.pa.us, ojford@gmail.com, peter@eisentraut.org, li.evan.chao@gmail.com, krasiyan@gmail.com, vik@postgresfriends.org, andrew@tao11.riddles.org.uk, david@fetter.org, pgsql-hackers@lists.postgresql.org
Date: 2025-10-07T20:14:27Z
Lists: pgsql-hackers

Commits

Same data as JSON: GET /api/v1/messages/:b64id/commits the thread's linked commits as JSON, with link sources. API reference →
  1. Fix coding style with "else".

  2. Fix multi WinGetFuncArgInFrame/Partition calls with IGNORE NULLS.

  3. Fix Coverity issue reported in commit 2273fa32bce.

  4. Use ereport rather than elog in WinCheckAndInitializeNullTreatment.

  5. Avoid uninitialized-variable warnings from older compilers.

  6. Fix Coverity issues reported in commit 25a30bbd423.

  7. Improve EXPLAIN's display of window functions.

  8. Automatically generate node support functions

On Mon, Oct 6, 2025 at 7:29 PM Tatsuo Ishii <ishii@postgresql.org> wrote:

>
> > Please disregard the v1 patch. It includes a bug: If
> > WinGetFuncArgInPartition() is called with set_mark == true and isout
> > == NULL, WinSetMarkPosition() is not called by
> > WinGetFuncArgInPartition().
> >
> > I will post v2 patch.
>
> Attached is the v2 patch.
>

Thanks! This passes regression, and reads right to my eye and (most
important to me) allows PostGIS to run under Pg19 again.

Thanks,
P