[very minor] Documentation suggestion
Hackorum builds and tests every patch posted to the lists, not only commitfest submissions. This is Hackorum's own CI rather than the PostgreSQL project's, and it is still under testing - please report anything that looks wrong.
The following documentation comment has been logged on the website:
Page: https://www.postgresql.org/docs/15/logicaldecoding-synchronous.html
Description:
Very minor suggestion. Earlier on this page, commands are referred to that
can cause "a deadlock". Later on the page, it says "cause deadlock".
https://www.postgresql.org/account/comments/new/15/logicaldecoding-synchronous.html/
"Note that these commands that can cause deadlock"
I think that may also be grammatically correct, however I thought "a
deadlock" (adding the "a") sounded smoother to me when reading. Suggestion
below (adds "a" before "deadlock").
"Note that these commands that can cause a deadlock"
On Fri, Oct 21, 2022 at 02:19:12AM +0000, PG Doc comments form wrote:
The following documentation comment has been logged on the website:
Page: https://www.postgresql.org/docs/15/logicaldecoding-synchronous.html
Description:Very minor suggestion. Earlier on this page, commands are referred to that
can cause "a deadlock". Later on the page, it says "cause deadlock".https://www.postgresql.org/account/comments/new/15/logicaldecoding-synchronous.html/
"Note that these commands that can cause deadlock"
The full sentence is:
Note that these commands that can cause deadlock apply to not only
explicitly indicated system catalog tables above but also to any other
[user] catalog table.
What I think it means is:
Note that these commands, that can cause deadlock, apply to not only
explicitly indicated system catalog tables above but also to any other
[user] catalog table.
so with "a" it would be:
Note that these commands, that can cause a deadlock, apply to not only
explicitly indicated system catalog tables above but also to any other
[user] catalog table.
but it still needs help ;-) Here is the text I just applied:
Note that these commands can cause deadlocks not only for the system
catalog tables listed above but for other catalog tables.
Attached is the applied patch to master.
--
Bruce Momjian <bruce@momjian.us> https://momjian.us
EDB https://enterprisedb.com
Only you can decide what is important to you.