Jeremy Finzel <finzelj@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