Incomplete sentence in the description for most_common_freqs

PG Doc comments form <noreply@postgresql.org> writes:

It seems the ending clarifying sentence:
"(Null when most_common_vals is.)"
should rather be:
"(Null when most_common_vals is null.)"

I think it's perfectly good English as-is, if a bit terse.

regards, tom lane

Thanks Tom!

True it is that is perfectly valid English. However, that interpretation might not result trivial to us non-native English speakers.

Would be great if at some point it could be changed to make it clearer.

BTW, it also appears in the definition of the pg_stats_ext_exprs system view.

Regards,
Nacho

________________________________
From: Tom Lane <tgl@sss.pgh.pa.us>
Sent: Wednesday, March 13, 2024 3:37 PM
To: Nacho Alonso Portillo <Ignacio.Alonso@microsoft.com>
Cc: pgsql-docs@lists.postgresql.org <pgsql-docs@lists.postgresql.org>
Subject: [EXTERNAL] Re: Incomplete sentence in the description for most_common_freqs

[You don't often get email from tgl@sss.pgh.pa.us. Learn why this is important at https://aka.ms/LearnAboutSenderIdentification ]

PG Doc comments form <noreply@postgresql.org> writes:

It seems the ending clarifying sentence:
"(Null when most_common_vals is.)"
should rather be:
"(Null when most_common_vals is null.)"

I think it's perfectly good English as-is, if a bit terse.

regards, tom lane