From: Tom Lane <tgl@sss.pgh.pa.us>
To: Tomas Vondra <tomas.vondra@2ndquadrant.com>
Cc: Alvaro Herrera <alvherre@2ndquadrant.com>
Cc: Simon Riggs <simon@2ndquadrant.com>
Cc: Andrew Dunstan <andrew.dunstan@2ndquadrant.com>
Cc: PostgreSQL-development <pgsql-hackers@postgresql.org>
Cc: Dean Rasheed <dean.a.rasheed@gmail.com>
Subject: Re: WITH clause in CREATE STATISTICS
Date: Fri, 12 May 2017 10:05:27 -0400
Message-ID: <17300.1494597927@sss.pgh.pa.us> (raw)
In-Reply-To: <b6b1f3a1-281a-20bc-d961-60604046c73d@2ndquadrant.com>
References: <20170511221330.5akgbsoyx6wm4u34@alvherre.pgsql>
<22676.1494557205@sss.pgh.pa.us>
<b6b1f3a1-281a-20bc-d961-60604046c73d@2ndquadrant.com>
List-Unsubscribe: <mailto:majordomo@postgresql.org?body=unsub%20pgsql-hackers>
Tomas Vondra <tomas.vondra@2ndquadrant.com> writes:
> On 5/12/17 4:46 AM, Tom Lane wrote:
>> If people agree that that reads better, we should make an effort to
>> propagate that terminology elsewhere in the docs, and into error messages,
>> objectaddress.c output, etc.
> I'm fine with the 'statistics object' wording. I've been struggling with
> this bit while working on the patch, and I agree it sounds better and
> getting it consistent is definitely worthwhile.
OK. We can work on that as a followon patch.
>> Although I've not done anything about it here, I'm not happy about the
>> handling of dependencies for stats objects.
> Yeah, it's a bit frankensteinian ...
I'm prepared to create a fix for that, but it'd be easier to commit the
current patch first, to avoid merge conflicts.
>> Lastly, I tried the example given in the CREATE STATISTICS reference page,
>> and it doesn't seem to work.
> I assume you're talking about the functional dependencies and in that
> case that's expected behavior, because f. dependencies do assume the
> conditions are "consistent" with the functional dependencies.
Hm. OK, but then that example is pretty misleading, because it leads
the reader to suppose that the planner can tell the difference between
the selectivities of the two queries. Maybe what's lacking is an
explanation of how you'd use this statistics type.
regards, tom lane
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
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: tgl@sss.pgh.pa.us, tomas.vondra@2ndquadrant.com, alvherre@2ndquadrant.com, simon@2ndquadrant.com, andrew.dunstan@2ndquadrant.com, dean.a.rasheed@gmail.com
Subject: Re: WITH clause in CREATE STATISTICS
In-Reply-To: <17300.1494597927@sss.pgh.pa.us>
* 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