On Fri, Apr 16, 2021 at 3:08 PM Japin Li <japi...@hotmail.com> wrote: > > On Fri, 16 Apr 2021 at 16:52, Amit Kapila <amit.kapil...@gmail.com> wrote: > > On Fri, Apr 16, 2021 at 12:55 PM Japin Li <japi...@hotmail.com> wrote: > > It is okay as POC but we can't change the existing function > > RelationGetIndexAttrBitmap. It is used from other places as well. It > > might be better to write a separate function for this, something like > > what Osumi-San's patch is trying to do. > > Agreed! > > Hi Osumi-San, can you merge the test case in your next version? >
+1. Your test looks reasonable to me. -- With Regards, Amit Kapila.
