On 10/03/2016 04:46 AM, Michael Paquier wrote:
On Fri, Sep 30, 2016 at 8:10 PM, Heikki Linnakangas <hlinn...@iki.fi> wrote:
This patch set is in pretty good shape, the only problem is that it's so big
that no-one seems to have the time or courage to do the final touches and
commit it.

Did you see my suggestions about simplifying its SQL structure? You
could shave some code without impacting the base set of features.

Yeah. The idea was to use something like pg_node_tree to store all the different kinds of statistics, the histogram, the MCV, and the functional dependencies, in one datum. Or JSON, maybe. It sounds better than an opaque bytea blob, although I'd prefer something more relational. For the functional dependencies, I think we could get away with a simple float array, so let's do that in the first cut, and revisit this for the MCV and histogram later. Separate columns for the functional dependencies, the MCVs, and the histogram, probably makes sense anyway.

- Heikki



--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to