From: | Alvaro Herrera <alvherre(at)commandprompt(dot)com> |
---|---|
To: | Boszormenyi Zoltan <zb(at)cybertec(dot)at> |
Cc: | pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, Hans-Juergen Schoenig <hs(at)cybertec(dot)at> |
Subject: | Re: TEXT vs PG_NODE_TREE in system columns (cross column and expression statistics patch) |
Date: | 2011-04-28 15:20:07 |
Message-ID: | 1304003607-sup-7783@alvh.no-ip.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Excerpts from Boszormenyi Zoltan's message of jue abr 28 11:03:56 -0300 2011:
> Hi,
>
> attached is the WIP patch for cross-column statistics and
> extra expression statistics.
>
> My question is that why pg_node_tree is unusable as
> syscache attribute? I attempted to alias it as text in the patch
> but I get the following error if I try to use it by setting
> USE_SYSCACHE_FOR_SEARCH to 1 in selfuncs.c.
> Directly using the underlying pg_statistic3 doesn't cause an error.
Two comments:
1. it seems that expression stats are mostly separate from cross-column
stats; does it really make sense to submit the two in the same patch?
2. there are almost no code comments anywhere
3. (bonus) if you're going to copy/paste pg_attribute.h verbatim into
the new files, please remove the bits you currently have in "#if 0".
(Not to mention the fact that the new catalogs seem rather poorly
named).
--
Álvaro Herrera <alvherre(at)commandprompt(dot)com>
The PostgreSQL Company - Command Prompt, Inc.
PostgreSQL Replication, Consulting, Custom Development, 24x7 support
From | Date | Subject | |
---|---|---|---|
Next Message | Alvaro Herrera | 2011-04-28 15:21:39 | Re: TEXT vs PG_NODE_TREE in system columns (cross column and expression statistics patch) |
Previous Message | Tom Lane | 2011-04-28 15:19:19 | Re: TEXT vs PG_NODE_TREE in system columns (cross column and expression statistics patch) |