Re: PoC Refactor AM analyse API
От | Jaime Casanova |
---|---|
Тема | Re: PoC Refactor AM analyse API |
Дата | |
Msg-id | 20210908150625.GA6483@ahch-to обсуждение исходный текст |
Ответ на | Re: PoC Refactor AM analyse API (Denis Smirnov <sd@arenadata.io>) |
Ответы |
Re: PoC Refactor AM analyse API
|
Список | pgsql-hackers |
On Fri, Feb 19, 2021 at 12:06:12PM +1000, Denis Smirnov wrote: > Thanks for your review, Heikki. > > I have made the changes you have requested. > > 1. All modifications interconnected with column projection were reverted (they should be added in https://commitfest.postgresql.org/31/2922if the current patch would be merged one day). > 2. I have returned PROGRESS_ANALYZE_* states. > 3. qsort() was moved into heapam_acquire_sample_rows(). Also a comment was added, that the acquire_sample_rows() AM functionmust return the tuples in a physical table order. > 4. table_beginscan_analyze() was removed as a redundant function. > 5. acquire_sample_rows() comment about reservoir was changed. > Hi Denis, This doesn't apply anymore because of c6fc50c, can you resubmit a new patch? Please note that the patch must be submitted with a .patch extension instead of .txt, that way the CI at http://commitfest.cputube.org/ will be able to execute automatic tests on it. Regards, -- Jaime Casanova Director de Servicios Profesionales SystemGuards - Consultores de PostgreSQL
В списке pgsql-hackers по дате отправления: