Re: IGNORE/RESPECT NULLS can be specified for (prokind == 'f').

Henson Choi <assam258@gmail.com>

From: Henson Choi <assam258@gmail.com>
To: Fujii Masao <masao.fujii@gmail.com>
Cc: Tatsuo Ishii <ishii@postgresql.org>, alvherre@kurilemu.de, li.evan.chao@gmail.com, jian.universality@gmail.com, pgsql-hackers@postgresql.org
Date: 2026-06-25T14:15:09Z
Lists: pgsql-hackers

Attachments

Hi Fujii-san,

I've pushed the patch. Thanks!


While reading ParseFuncOrColumn(), I noticed that commit 419ce13b701
left a stray blank line just before the closing brace of the
normal-aggregate block in the FUNCDETAIL_AGGREGATE arm (right before
the new "aggregate functions do not accept RESPECT/IGNORE NULLS"
check).

Trivial cleanup attached; no functional change.

Regards,
Henson Choi

Commits

  1. Remove stray blank line in ParseFuncOrColumn()

  2. Refine error reporting for null treatment on non-window functions

  3. Fix error message typo.

  4. Fix to not allow null treatment to non window functions.

  5. Add IGNORE NULLS/RESPECT NULLS option to Window functions.