pg.ddx.io pgsql-hackers@postgresql.org mailing list archive help / color / mirror / Atom feed From: Tomas Vondra <tomas.vondra@2ndquadrant.com>
To: Dean Rasheed <dean.a.rasheed@gmail.com>
Cc: David Rowley <david.rowley@2ndquadrant.com>
Cc: Kyotaro HORIGUCHI <horiguchi.kyotaro@lab.ntt.co.jp>
Cc: Alvaro Herrera <alvherre@2ndquadrant.com>
Cc: Michael Paquier <michael@paquier.xyz>
Cc: Andres Freund <andres@anarazel.de>
Cc: Thomas Munro <thomas.munro@enterprisedb.com>
Cc: Bruce Momjian <bruce@momjian.us>
Cc: Mark Dilger <hornschnorter@gmail.com>
Cc: PostgreSQL Hackers <pgsql-hackers@postgresql.org>
Subject: Re: [HACKERS] PATCH: multivariate histograms and MCV lists
Date: Wed, 27 Mar 2019 20:55:07 +0100
Message-ID: <20190327195507.GA21957@development> (raw )
In-Reply-To: <20190326152108.GB3088@development >
References: <37ce47c5-e0cf-c6d3-6923-6b8cfd1cc6b1@2ndquadrant.com >
<CAEZATCXfk-Syby-MWLFfN-v=7HFEm-QYyX+tP=NVZ+iNw8ELUw@mail.gmail.com >
<c08dd2ce-14f0-463b-763c-13af49926d03@2ndquadrant.com >
<CAKJS1f_naKM6kzeUdDs61=++fYdtsk4T5586DZObRvUXPVmL9Q@mail.gmail.com >
<fd4c5223-6c1a-c2d5-c0e0-53db789a0bdc@2ndquadrant.com >
<CAKJS1f9o23x-utM3Jk2wji_tQR-CZgyPP5cqQgt1e_zhHMYQpw@mail.gmail.com >
<add2f0d3-4fbc-af7c-e02e-0f2b2d6dedc3@2ndquadrant.com >
<CAEZATCURhZv0kMsy9PoWvx+4nuvmfGi42f9kte7ZNyWOJRKKjw@mail.gmail.com >
<CAEZATCV=W+9LKu=BVbBAgOn3GChGcm=Qdhgco3e3aDxkRqfcVg@mail.gmail.com >
<20190326152108.GB3088@development >
Hi,
I've now committed the MCV part, after addressing the last two issues
raised by Dean:
* The MCV build now always uses the mincount to decide which of the
items to keep in the list.
* Both the MCV build and deserialization now uses the same maximum number
of list items (10k).
Unfortunately, I forgot to merge these two fixes before pushing, so I had
to commit them separately. Sorry about that :/
Attached are the remaining parts of this patch series - the multivariate
histograms, and also a new patch tweaking regression tests for the old
statistic types (ndistinct, dependencies) to adopt the function-based
approach instead of the regular EXPLAIN.
But those are clearly a matter for the future (well, maybe it'd make sense
to commit the regression test change now).
regards
--
Tomas Vondra http://www.2ndQuadrant.com
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
Attachments:
[application/gzip] 0001-adopt-regression-tests.patch.gz (3.1K, ../20190327195507.GA21957@development/2-0001-adopt-regression-tests.patch.gz)
download
[application/gzip] 0002-multivariate-histograms.patch.gz (48.4K, ../20190327195507.GA21957@development/3-0002-multivariate-histograms.patch.gz)
download
view thread (167+ messages) latest in thread
Message-ID: <20190327195507.GA21957@development>
Permalink: ../20190327195507.GA21957@development/
Also on: postgresql.org/message-id/20190327195507.GA21957@development
copy link · copy postgr.es
reply Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-hackers@postgresql.org
Cc: tomas.vondra@2ndquadrant.com, dean.a.rasheed@gmail.com, david.rowley@2ndquadrant.com, horiguchi.kyotaro@lab.ntt.co.jp, alvherre@2ndquadrant.com, michael@paquier.xyz, andres@anarazel.de, thomas.munro@enterprisedb.com, bruce@momjian.us, hornschnorter@gmail.com
Subject: Re: [HACKERS] PATCH: multivariate histograms and MCV lists
In-Reply-To: <20190327195507.GA21957@development>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox