[very minor] Documentation suggestion

Started by PG Bug reporting formover 3 years ago2 messagesdocs
Jump to latest
#1PG Bug reporting form
noreply@postgresql.org

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"

#2Bruce Momjian
bruce@momjian.us
In reply to: PG Bug reporting form (#1)
Re: [very minor] Documentation suggestion

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.

Attachments:

master.difftext/x-diff; charset=us-asciiDownload+2-3