Hi, Soumyadeep: Happy New Year. +typedef struct neededColumnContext +{ + Bitmapset **mask; + int n;
+ * n specifies the number of allowed entries in mask: we use + * it for bounds-checking in the walker above. I think the code would be easier to read if the above comment is moved or copied for field n of neededColumnContext Cheers On Thu, Dec 31, 2020 at 1:03 PM Soumyadeep Chakraborty < soumyadeep2...@gmail.com> wrote: > Hey Masahiko, > > I added it to the Jan CF (https://commitfest.postgresql.org/31/2922/). > > PFA a rebased version against latest head. > > Regards, > Soumyadeep >