Thread: Re: Fix typo in commens on ExecInsertIndexTuples

Re: Fix typo in commens on ExecInsertIndexTuples

From
Amit Langote
Date:
Hi Nagata-san,

On Tue, Jul 1, 2025 at 11:02 AM Yugo Nagata <nagata@sraoss.co.jp> wrote:
>
> Hi,
>
> I found typos in the comments of ExecInsertIndexTuples
> that mention TUUI_{All, Summarizing} but they should be
> TU_*.
>
> I've attached a patch to fix it.

Good catch and thanks for the patch.  Will push shortly.

--
Thanks, Amit Langote



Re: Fix typo in commens on ExecInsertIndexTuples

From
Amit Langote
Date:
On Tue, Jul 1, 2025 at 11:42 AM Amit Langote <amitlangote09@gmail.com> wrote:
> Hi Nagata-san,
>
> On Tue, Jul 1, 2025 at 11:02 AM Yugo Nagata <nagata@sraoss.co.jp> wrote:
> >
> > Hi,
> >
> > I found typos in the comments of ExecInsertIndexTuples
> > that mention TUUI_{All, Summarizing} but they should be
> > TU_*.
> >
> > I've attached a patch to fix it.
>
> Good catch and thanks for the patch.  Will push shortly.

Done and backpatched to v16, where the faulty comments were introduced.

--
Thanks, Amit Langote



Re: Fix typo in commens on ExecInsertIndexTuples

From
Yugo Nagata
Date:
On Tue, 1 Jul 2025 13:29:44 +0900
Amit Langote <amitlangote09@gmail.com> wrote:

> On Tue, Jul 1, 2025 at 11:42 AM Amit Langote <amitlangote09@gmail.com> wrote:
> > Hi Nagata-san,
> >
> > On Tue, Jul 1, 2025 at 11:02 AM Yugo Nagata <nagata@sraoss.co.jp> wrote:
> > >
> > > Hi,
> > >
> > > I found typos in the comments of ExecInsertIndexTuples
> > > that mention TUUI_{All, Summarizing} but they should be
> > > TU_*.
> > >
> > > I've attached a patch to fix it.
> >
> > Good catch and thanks for the patch.  Will push shortly.
> 
> Done and backpatched to v16, where the faulty comments were introduced.

Thanks!

Regards,
Yugo Nagata

-- 
Yugo Nagata <nagata@sraoss.co.jp>