pg.ddx.io  pgsql-committers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Dean Rasheed <dean.a.rasheed@gmail.com>
To: pgsql-committers@lists.postgresql.org
Subject: pgsql: doc: improve description of privileges for MERGE and update glos
Date: Sat, 18 Nov 2023 12:44:11 +0000
Message-ID: <E1r4KgF-00639h-DU@gemulon.postgresql.org> (raw)

doc: improve description of privileges for MERGE and update glossary.

On the MERGE page, the description of the privileges required could be
taken to imply that the SELECT privilege is required on all columns of
the data source, whereas actually it is only required on the columns
referred to by conditions or expressions in the MERGE command. Re-word
it to make that a little clearer, and mention expressions as well as
conditions.

Also, add a glossary entry for MERGE, and nearby on the glossary page,
mention MERGE in the list of commands that cannot update a
materialized view.

Noted by Jian He. Patch by me, reviewed by Jian He.

Discussion: https://postgr.es/m/CACJufxHuSoRXKwr0MtSFLXuT2nFVWcVfEWhxg7qdP9h%2Bs3a%2BUw%40mail.gmail.com

Branch
------
REL_15_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/ff772853d02e274684567620fba56dd1f10f2489

Modified Files
--------------
doc/src/sgml/glossary.sgml  | 21 +++++++++++++++++++--
doc/src/sgml/ref/merge.sgml | 10 +++++-----
2 files changed, 24 insertions(+), 7 deletions(-)



view thread (3+ messages)  latest in thread

Message-ID: <E1r4KgF-00639h-DU@gemulon.postgresql.org>
Permalink:  ../E1r4KgF-00639h-DU@gemulon.postgresql.org/
Also on:    postgresql.org/message-id/E1r4KgF-00639h-DU@gemulon.postgresql.org

 · 

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-committers@postgresql.org
  Cc: dean.a.rasheed@gmail.com, pgsql-committers@lists.postgresql.org
  Subject: Re: pgsql: doc: improve description of privileges for MERGE and update glos
  In-Reply-To: <E1r4KgF-00639h-DU@gemulon.postgresql.org>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox