Jeremy Finzel <finz...@gmail.com> writes:
> I've not submitted a patch before, and have a few suggestions I'd like
> feedback on before I write one (for the docs only).

OK ...

> First, even this summary looks untrue:
> REFRESH MATERIALIZED VIEW — replace the contents of a materialized view.

Agreed.  I'd just make it say "update the contents...", personally.
More words are not better in command summaries.

> Also, the Description part has the same inaccuracy:
> "completely replaces the contents of a materialized view.....The old
> contents are discarded."

Yeah, that just wasn't updated :-(

> On a different note, none of the documentation on materialized views notes
> that they can only be LOGGED.  This should be noted,

Also agreed.

                        regards, tom lane


Reply via email to