| From: | Tomas Vondra <tomas(dot)vondra(at)2ndquadrant(dot)com> |
|---|---|
| To: | Dean Rasheed <dean(dot)a(dot)rasheed(at)gmail(dot)com> |
| Cc: | David Rowley <david(dot)rowley(at)2ndquadrant(dot)com>, Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, Michael Paquier <michael(at)paquier(dot)xyz>, Andres Freund <andres(at)anarazel(dot)de>, Thomas Munro <thomas(dot)munro(at)enterprisedb(dot)com>, Bruce Momjian <bruce(at)momjian(dot)us>, Mark Dilger <hornschnorter(at)gmail(dot)com>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: [HACKERS] PATCH: multivariate histograms and MCV lists |
| Date: | 2019-03-13 01:25:40 |
| Message-ID: | 19f76496-dcf3-ccea-dd82-26fbed57b8f5@2ndquadrant.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hi,
attached is an updated version of the patch, addressing most of the
issues raised in the recent reviews. There are two main exceptions:
1) I haven't reworked the regression tests to use a function to check
cardinality estimates and making them faster.
2) Review handling of bitmap in statext_is_compatible_clause_internal
when processing AND/OR/NOT clauses.
I plan to look into those items next, but I don't want block review of
other parts of the patch unnecessarily.
regards
--
Tomas Vondra http://www.2ndQuadrant.com
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
| Attachment | Content-Type | Size |
|---|---|---|
| 0001-multivariate-MCV-lists-20190312.patch.gz | application/gzip | 39.8 KB |
| 0002-multivariate-histograms-20190312.patch.gz | application/gzip | 49.0 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Thomas Munro | 2019-03-13 01:27:47 | Re: Refactoring the checkpointer's fsync request queue |
| Previous Message | Shawn Debnath | 2019-03-13 01:00:07 | Re: Refactoring the checkpointer's fsync request queue |