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