From: | Masahiko Sawada <masahiko(dot)sawada(at)2ndquadrant(dot)com> |
---|---|
To: | Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> |
Cc: | PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: Fix typos in reorderbuffer.c |
Date: | 2020-10-08 09:09:24 |
Message-ID: | CA+fd4k4LjT6z_Qqongm1U4DnW8ej90C-mcQDuJw3RuL=XkCtyA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Thu, 8 Oct 2020 at 17:37, Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> wrote:
>
> @@ -1432,7 +1432,7 @@ ReorderBufferCleanupTXN(ReorderBuffer *rb,
> ReorderBufferTXN *txn)
> ReorderBufferCleanupTXN(rb, subtxn);
> }
>
> - /* cleanup changes in the toplevel txn */
> + /* cleanup changes in the txn */
> dlist_foreach_modify(iter, &txn->changes)
> {
> ReorderBufferChange *change;
> @@ -1533,7 +1533,7 @@ ReorderBufferTruncateTXN(ReorderBuffer *rb,
> ReorderBufferTXN *txn)
> ReorderBufferTruncateTXN(rb, subtxn);
> }
>
> - /* cleanup changes in the toplevel txn */
> + /* cleanup changes in the txn */
> dlist_foreach_modify(iter, &txn->changes)
> {
> ReorderBufferChange *change;
>
> Both the above functions are recursive and will clean the changes for
> both the top-level transaction and subtransactions.
Right.
> So, I feel the
> comments should be accordingly updated.
+1 for this change.
Regards,
--
Masahiko Sawada http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
From | Date | Subject | |
---|---|---|---|
Next Message | k.jamison@fujitsu.com | 2020-10-08 09:13:48 | RE: [Patch] Optimize dropping of relation buffers using dlist |
Previous Message | Dilip Kumar | 2020-10-08 09:08:27 | Re: [HACKERS] Custom compression methods |