Hi Tatsuo,
> BTW, I plan to add a note to "9.23. Window Functions" in the
> PostgreSQL documentation, something like below to explicitly stat
> that the patch does not support the RUNNING keyword. Comments?
Looks good to me. Just one sentence to suggest:
> The SQL standard allows to put a keyword RUNNING in front of row
> pattern navigation functions FIRST and LAST.
"allows to put" reads awkwardly in English. How about matching the
first sentence of the nth_value note just above it ("The SQL standard
defines a FROM FIRST or FROM LAST option for nth_value."):
The SQL standard defines a RUNNING prefix for the row pattern
navigation functions FIRST and LAST.
Then the two notes also continue the same way, with "This is not
implemented in PostgreSQL." The rest reads fine as is.
Best regards,
Henson