Re: extended statistics: n-distinct
Alvaro Herrera <alvherre@2ndquadrant.com>
From: Alvaro Herrera <alvherre@2ndquadrant.com>
To: Pg Hackers <pgsql-hackers@postgresql.org>
Cc: Tomas Vondra <tomas.vondra@2ndquadrant.com>, Dean Rasheed <dean.a.rasheed@gmail.com>, Kyotaro HORIGUCHI <horiguchi.kyotaro@lab.ntt.co.jp>
Date: 2017-03-20T22:19:05Z
Lists: pgsql-hackers
Attachments
- split-analyze-h.patch (text/plain) patch
Alvaro Herrera wrote: > * I'm not terribly happy with the header organization. I think > VacAttrStats should be in its own (new) src/include/statistics/analyze.h > for example (which cleans up a bunch of existing stuff a bit) I tried this and it doesn't actually do any good. Patch attached, which I intend to throw away. The main problem is that most files interested in analyze stuff also wants to do vacuum_delay_point, or access default_statistics_target, both of which are declared in vacuum.h, so these files have to include vacuum.h anyway. In order to achieve anything worthwhile we'd have to do a three-way split I think, and that's more trouble that this is worth. -- Álvaro Herrera https://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
Commits
-
Implement multivariate n-distinct coefficients
- 7b504eb282ca 10.0 landed