On 2022-Jun-01, Justin Pryzby wrote:
> I prefer that way, with "See also" after the text that requires more
> information. But the most important thing is to include the link at all.
But it's not a "see also". It's a link to the primary source of
concurrency information for MERGE. The text that follows is not talking
about concurrency, it just indicates that you can do something different
but related by using a different command.
Re-reading the modified paragraph, I propose "see X for a thorough
explanation on the behavior of MERGE under concurrency". However, in
the proposed patch the link goes to Chapter 13 "Concurrency Control",
and the explanation that we intend to link to is hidden in subsection
13.2.1 "Read Committed Isolation level". So it appears that we do not
have any explanation on how MERGE behaves in other isolation levels.
That can't be good ...
--
Álvaro Herrera Breisgau, Deutschland — https://www.EnterpriseDB.com/
"Cuando mañana llegue pelearemos segun lo que mañana exija" (Mowgli)
