On Sun, Nov 01, 2020 at 09:23:44AM +0900, Michael Paquier wrote: > By doing so, there is no need to include pg_statistic.h in index.c. > Except that, the logic looks fine at quick glance. In the long-term, > I also think that it would make sense to move both routnes out of > heap.c into a separate pg_statistic.c. That's material for a separate > patch of course.
I have looked again at that, and applied it after some tweaks. Particularly, I did not really like the use of "old" and "new" for the copy from the old to a new relation in the new function, so I have replaced that by "from" and "to". -- Michael
signature.asc
Description: PGP signature