obsolete comment in ExecBRUpdateTriggers()

Started by Amit Langoteabout 6 years ago2 messages
#1Amit Langote
amitlangote09@gmail.com
1 attachment(s)

Hi,

It seems that d986d4e87f6 forgot to update a comment upon renaming a variable.

Attached fixes it.

Thanks,
Amit

Attachments:

ExecBRUpdateTriggers-obsolete-commment.patchtext/plain; charset=US-ASCII; name=ExecBRUpdateTriggers-obsolete-commment.patchDownload
diff --git a/src/backend/commands/trigger.c b/src/backend/commands/trigger.c
index faeea16d21..99cb5bf557 100644
--- a/src/backend/commands/trigger.c
+++ b/src/backend/commands/trigger.c
@@ -3033,8 +3033,8 @@ ExecBRUpdateTriggers(EState *estate, EPQState *epqstate,
 		/*
 		 * In READ COMMITTED isolation level it's possible that target tuple
 		 * was changed due to concurrent update.  In that case we have a raw
-		 * subplan output tuple in newSlot, and need to run it through the
-		 * junk filter to produce an insertable tuple.
+		 * subplan output tuple in epqslot_candidate, and need to run it
+		 * through the junk filter to produce an insertable tuple.
 		 *
 		 * Caution: more than likely, the passed-in slot is the same as the
 		 * junkfilter's output slot, so we are clobbering the original value
#2Robert Haas
robertmhaas@gmail.com
In reply to: Amit Langote (#1)
Re: obsolete comment in ExecBRUpdateTriggers()

On Wed, Dec 18, 2019 at 1:49 AM Amit Langote <amitlangote09@gmail.com> wrote:

It seems that d986d4e87f6 forgot to update a comment upon renaming a variable.

Attached fixes it.

Committed and back-patched to v12.

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company