From: Justin Pryzby <pryzby@telsasoft.com>
To: Tomas Vondra <tomas.vondra@enterprisedb.com>
Cc: Dean Rasheed <dean.a.rasheed@gmail.com>
Cc: hackers <pgsql-hackers@postgresql.org>
Subject: Re: PoC/WIP: Extended statistics on expressions
Date: Wed, 1 Sep 2021 14:38:16 -0500
Message-ID: <20210901193816.GO26465@telsasoft.com> (raw)
In-Reply-To: <6bc0ab0c-a807-2b7f-7df8-ac90c7839013@enterprisedb.com>
References: <20201123022604.GD24052@telsasoft.com>
<e5beb5d5-28b4-7aad-3a16-9a6abf57f897@enterprisedb.com>
<6455e0ad-2a2e-bdb0-e1f8-fd4983d1c8aa@enterprisedb.com>
<087086de-1384-8374-4efc-2e7915071d5c@enterprisedb.com>
<CAEZATCU9uPo7JYdx4k0-ufXXZH8t7itodibUwCva+s+AvAKcnw@mail.gmail.com>
<b2995773-c9a1-5d3b-fc90-4f3ab189be11@enterprisedb.com>
<20210816013255.GS10479@telsasoft.com>
<a2be7e05-e6b7-1467-d15a-49011e45a4b8@enterprisedb.com>
<20210824131320.GO10479@telsasoft.com>
<6bc0ab0c-a807-2b7f-7df8-ac90c7839013@enterprisedb.com>
On Wed, Sep 01, 2021 at 06:45:29PM +0200, Tomas Vondra wrote:
> > > Patch 0001 fixes the "double parens" issue discussed elsewhere in this
> > > thread, and patch 0002 tweaks CREATE STATISTICS to treat "(a)" as a simple
> > > column reference.
> >
> > 0002 refuses to create expressional stats on a simple column reference like
> > (a), which I think is helps to avoid a user accidentally creating useless ext
> > stats objects (which are redundant with the table's column stats).
> >
> > 0002 does not attempt to refuse cases like (a+0), which I think is fine:
> > we don't try to reject useless cases if someone insists on it.
> > See 240971675, 701fd0bbc.
> >
> > So I am +1 to apply both patches.
> >
> > I added this as an Opened Item for increased visibility.
>
> I've pushed both fixes, so the open item should be resolved.
Thank you - I marked it as such.
There are some typos in 537ca68db (refenrece)
I'll add them to my typos branch if you don't want to patch them right now or
wait to see if someone notices anything else.
diff --git a/src/backend/commands/statscmds.c b/src/backend/commands/statscmds.cindex 59369f8736..17cbd97808 100644--- a/src/backend/commands/statscmds.c+++ b/src/backend/commands/statscmds.c@@ -205,27 +205,27 @@ CreateStatistics(CreateStatsStmt *stmt)
numcols = list_length(stmt->exprs);
if (numcols > STATS_MAX_DIMENSIONS)
ereport(ERROR,
(errcode(ERRCODE_TOO_MANY_COLUMNS),
errmsg("cannot have more than %d columns in statistics",
STATS_MAX_DIMENSIONS)));
/*
* Convert the expression list to a simple array of attnums, but also keep
* a list of more complex expressions. While at it, enforce some
* constraints - we don't allow extended statistics on system attributes,
- * and we require the data type to have less-than operator.+ * and we require the data type to have a less-than operator.
*
- * There are many ways how to "mask" a simple attribute refenrece as an+ * There are many ways to "mask" a simple attribute reference as an
* expression, for example "(a+0)" etc. We can't possibly detect all of
- * them, but we handle at least the simple case with attribute in parens.+ * them, but we handle at least the simple case with the attribute in parens.
* There'll always be a way around this, if the user is determined (like
* the "(a+0)" example), but this makes it somewhat consistent with how
* indexes treat attributes/expressions.
*/
foreach(cell, stmt->exprs)
{
StatsElem *selem = lfirst_node(StatsElem, cell);
if (selem->name) /* column reference */
{
char *attname;
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: pryzby@telsasoft.com, tomas.vondra@enterprisedb.com, dean.a.rasheed@gmail.com
Subject: Re: PoC/WIP: Extended statistics on expressions
In-Reply-To: <20210901193816.GO26465@telsasoft.com>
* 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