Re: first_value()/nth_value() with IGNORE NULLS loop forever (EXCLUDE GROUP/TIES, frame past partition end, no ORDER BY)

Hi Shihao,

Fixing typo in the previous message...

>> Hi Bhuvan,
>>
>> Thanks for the report and the patch. I can reproduce all five queries
>> on master and on 19rc1.
>>
>> This came in with 25a30bbd423. I think it belongs on the 19 open items.
>>
>> https://wiki.postgresql.org/wiki/PostgreSQL_19_Open_Items
>
> Yeah, I have just added this to the 19 open items.
>
>> With your patch all five return at once and the regression tests
>> pass. I also compared the three functions against array_agg(x) FILTER
>> (WHERE x IS NOT NULL) over about 11k frame combinations. That included
>> spill to disk and volatile arguments. No differences.
>>
>> One suggestion for the test. Could you add last_value IGNORE NULLS
>> with EXCLUDE TIES and no ORDER BY? It scans backward, so it would
>> catch a later fix in row_is_in_frame that returns -1 for every
>> position after the current row.
>
> I have just started to read the discussion and patches. Just to
> clarify, is the test for feature improvement of row_is_in_frame?
~~~~~~~future

Regards,
--
Tatsuo Ishii
SRA OSS K.K.
English: http://www.sraoss.co.jp/index_en/
Japanese:http://www.sraoss.co.jp