On Mon, Dec 17, 2018 at 10:44 AM Michael Paquier <michael@paquier.xyz> wrote:
>
> On Fri, Dec 14, 2018 at 08:08:45AM +0000, Amul Sul wrote:
> > dump-alter-index-stats-v2.patch looks pretty much reasonable to me, passing on committer.
> >
> > The new status of this patch is: Ready for Committer
>
> Thanks Amul for the review. I got the occasion to look again at this
> patch, and I have read again the original thread which has added the new
> grammar for ALTER INDEX SET STATISTICS:
> https://www.postgresql.org/message-id/CAPpHfdsSYo6xpt0F=ngAdqMPFJJhC7zApde9h1qwkdpHpwFisA@mail.gmail.com
>
> As Alexander and others state on this thread, it looks a bit weird to
> use internally-produced attribute names in those SQL queries, which is
> why the new grammar has been added. At the same time, it looks more
> solid to me to represent the dumps with those column names instead of
> column numbers. Tom, Alexander, as you have commented on the original
> thread, perhaps you have an opinion here to share?
>
Oh I see -- understood the problem, I missed this discussion, thanks to
letting me know.
> For now, attached is an updated patch which has a simplified test list
> in the TAP test. I have also added two free() calls for the arrays
> getting allocated when statistics are present for an index.
Patch is missing?
Regards,
Amul
