On Wed, Dec 18, 2019 at 3:36 PM Amit Kapila <amit.kapil...@gmail.com> wrote: > > Few other comments which I have not fixed: >
+ /* interface function to support parallel vacuum */ + amestimateparallelvacuum_function amestimateparallelvacuum; /* can be NULL */ } IndexAmRoutine; One more thing, why have you removed the estimate function for API patch? It seems to me Robert has given a different suggestion [1] to deal with it. I think he suggests to add a new member like void *private_data to IndexBulkDeleteResult and then provide an estimate function. See his email [1] for detailed explanation. Did I misunderstood it or you have handled it differently? Can you please share your thoughts on this? [1] - https://www.postgresql.org/message-id/CA%2BTgmobjtHdLfQhmzqBNt7VEsz%2B5w3P0yy0-EsoT05yAJViBSQ%40mail.gmail.com -- With Regards, Amit Kapila. EnterpriseDB: http://www.enterprisedb.com