Re: Implement <null treatment> for window functions

> On 30 Jun 2020, at 15:54, Vik Fearing  wrote:

> This feature adds RESPECT NULLS and IGNORE NULLS syntax to several
> window functions, according to the SQL Standard.

This fails compilation due to a compiler warning in WinGetFuncArgInPartition
and WinGetFuncArgInFrame (same warning in both):

nodeWindowAgg.c: In function ‘WinGetFuncArgInPartition’:
nodeWindowAgg.c:3274:10: error: ‘step’ may be used uninitialized in this function [-Werror=maybe-uninitialized]
   relpos += step;
          ^
This was with GCC in the Travis build, the Windows build passed and so does
clang locally for me.

cheers ./daniel