pg.ddx.io pgsql-committers@postgresql.org mailing list archivehelp / color / mirror / Atom feed
pgsql: doc: improve description of privileges for MERGE and update glos 3+ messages / 1 participants [nested] [flat]
* pgsql: doc: improve description of privileges for MERGE and update glos @ 2023-11-18 12:44 Dean Rasheed <dean.a.rasheed@gmail.com> 0 siblings, 0 replies; 3+ messages in thread From: Dean Rasheed @ 2023-11-18 12:44 UTC (permalink / raw) To: pgsql-committers@lists.postgresql.org 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(-) ^ permalink raw reply [nested|flat] 3+ messages in thread
* pgsql: doc: improve description of privileges for MERGE and update glos @ 2023-11-18 12:44 Dean Rasheed <dean.a.rasheed@gmail.com> 0 siblings, 0 replies; 3+ messages in thread From: Dean Rasheed @ 2023-11-18 12:44 UTC (permalink / raw) To: pgsql-committers@lists.postgresql.org 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 ------ master Details ------- https://git.postgresql.org/pg/commitdiff/4bc8f29088f8a2e9c89e6c0819337c4c8bf5f20a Modified Files -------------- doc/src/sgml/glossary.sgml | 21 +++++++++++++++++++-- doc/src/sgml/ref/merge.sgml | 10 +++++----- 2 files changed, 24 insertions(+), 7 deletions(-) ^ permalink raw reply [nested|flat] 3+ messages in thread
* pgsql: doc: improve description of privileges for MERGE and update glos @ 2023-11-18 12:44 Dean Rasheed <dean.a.rasheed@gmail.com> 0 siblings, 0 replies; 3+ messages in thread From: Dean Rasheed @ 2023-11-18 12:44 UTC (permalink / raw) To: pgsql-committers@lists.postgresql.org 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_16_STABLE Details ------- https://git.postgresql.org/pg/commitdiff/3b6728910ace14c64ffa43b538a84508f4fe6408 Modified Files -------------- doc/src/sgml/glossary.sgml | 21 +++++++++++++++++++-- doc/src/sgml/ref/merge.sgml | 10 +++++----- 2 files changed, 24 insertions(+), 7 deletions(-) ^ permalink raw reply [nested|flat] 3+ messages in thread
end of thread, other threads:[~2023-11-18 12:44 UTC | newest] Thread overview: 3+ messages (download: mbox mbox.gz follow: Atom feed) -- links below jump to the message on this page -- 2023-11-18 12:44 pgsql: doc: improve description of privileges for MERGE and update glos Dean Rasheed <dean.a.rasheed@gmail.com> 2023-11-18 12:44 pgsql: doc: improve description of privileges for MERGE and update glos Dean Rasheed <dean.a.rasheed@gmail.com> 2023-11-18 12:44 pgsql: doc: improve description of privileges for MERGE and update glos Dean Rasheed <dean.a.rasheed@gmail.com>
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