Amit Kapila <amit.kapila16@gmail.com> writes:
> On Wed, Sep 14, 2022 at 8:16 AM Kyotaro Horiguchi
> <horikyota.ntt@gmail.com> wrote:
>> I found a small typo in a comment in pgbench.c of 15/master.
>> - * Return the number fo failed transactions.
>> + * Return the number of failed transactions.
> LGTM.
+1
>> While at it, I found "* lot fo unnecessary work." in pg13's
>> procsignal.c. It has been fixed by 2a093355aa in PG14 but PG13 was
>> left alone at the time.
> I think sometimes we fix typos only in HEAD. I am not sure if we have
> a clear policy to backpatch such things.
I would not go back and change v13 at this point. You're right
that this is fuzzy, but overriding the contemporaneous decision
not to backpatch seems well outside our usual habits.
There are basically two good reasons to back-patch comment changes:
* fear that the comment is wrong enough to mislead people looking
at the older branch;
* fear that leaving it alone will create a merge hazard for future
back-patches.
It doesn't seem to me that either of those is a strong concern
in this case. In the absence of these concerns, back-patching
seems like make-work (and useless expenditure of buildfarm
cycles).
regards, tom lane
