Обсуждение: Adjust comments for `IndexOptInfo` to accurately reflect indexcollations's length

Hi,

Length of indexcollations[] in `IndexOptInfo` is nkeycolumns, but
struct comments fail to express this, it says:

indexkeys[], indexcollations[] each have ncolumns entries.

So we should polish this description. trivial patch attached.

-- 
Regards
Junwang Zhao

On Sun, 23 Nov 2025 at 01:44, Junwang Zhao <zhjwpku@gmail.com> wrote:
> Length of indexcollations[] in `IndexOptInfo` is nkeycolumns, but
> struct comments fail to express this, it says:
>
> indexkeys[], indexcollations[] each have ncolumns entries.
>
> So we should polish this description. trivial patch attached.

Thanks. I agree with the patch.  We should probably mention
canreturn[] too. I can take care of this.

David

On Mon, Nov 24, 2025 at 6:52 AM David Rowley <dgrowleyml@gmail.com> wrote:
>
> On Sun, 23 Nov 2025 at 01:44, Junwang Zhao <zhjwpku@gmail.com> wrote:
> > Length of indexcollations[] in `IndexOptInfo` is nkeycolumns, but
> > struct comments fail to express this, it says:
> >
> > indexkeys[], indexcollations[] each have ncolumns entries.
> >
> > So we should polish this description. trivial patch attached.
>
> Thanks. I agree with the patch.  We should probably mention
> canreturn[] too. I can take care of this.

Agreed. Thanks for taking care of this.

>
> David



--
Regards
Junwang Zhao

On Mon, 24 Nov 2025 at 14:57, Junwang Zhao <zhjwpku@gmail.com> wrote:
>
> On Mon, Nov 24, 2025 at 6:52 AM David Rowley <dgrowleyml@gmail.com> wrote:
> >
> > On Sun, 23 Nov 2025 at 01:44, Junwang Zhao <zhjwpku@gmail.com> wrote:
> > Thanks. I agree with the patch.  We should probably mention
> > canreturn[] too. I can take care of this.
>
> Agreed. Thanks for taking care of this.

Pushed.

David