Re: Some message fixes

Kyotaro HORIGUCHI wrote:

> 1. some messages are missing partitioned table/index
>
> The first attached. I'm not sure how the orering ought to be
> but I arranged them in mainly in the appearance order in if()
> conditions, or the order of case label in switch()
> construct. One exception is ATExecChangeOwner, in which the
> order is the others regardless of the case label order just
> above.
>
> This is backpatchable to 10 but 10 doesn't have partitioned
> index so needs a different fix (Second attached).
>
> # But, I'm not sure if the list may be that long...

I *think* the idea here is that a partitioned table is a table, so there
is no need to say "foo is not a table or partitioned table". We only
mention partitioned tables when we want to make a distinction between
those that are partitioned and those that aren't.

Same with indexes.

> 2. GUC comment for autovacuum_work_mem has a typo, "max num of
> parallel workers *than* can be..". (Third attached)

Yeah, pushed, though it's parallel_max_workers not autovacuum_work_mem.

--
Álvaro Herrera https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services