| From: | "Brendan Jurd" <direvus(at)gmail(dot)com> |
|---|---|
| To: | "Jeff Davis" <pgsql(at)j-davis(dot)com> |
| Cc: | "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us>, "Martijn van Oosterhout" <kleptog(at)svana(dot)org>, pgsql-hackers(at)postgresql(dot)org, npboley(at)gmail(dot)com |
| Subject: | Re: new correlation metric |
| Date: | 2008-11-03 07:33:01 |
| Message-ID: | 37ed240d0811022333w1cd4bec8v9952a074a819d8ce@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hi Jeff,
I've been assigned to do an initial review of your "new correlation
metric" patch.
If I'm grokking the thread, it looks like Tom suggested a substantial
change in the approach (targetting per-index correlation rather than
per-column) [1], and although you agreed with the spirit of his
suggestion[2], there hasn't been a new version of the patch submitted
in response.
The result is, I'm not really sure how I should direct my effort here.
Is it worth me reviewing the patch as it stands, or should I hold off
until a new version has been prepared, incorporating Tom's comments?
Cheers,
BJ
[1] http://archives.postgresql.org/pgsql-hackers/2008-10/msg01284.php
[2] http://archives.postgresql.org/pgsql-hackers/2008-10/msg01287.php
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Dimitri Fontaine | 2008-11-03 08:55:03 | Re: WIP patch: convert SQL-language functions to return tuplestores |
| Previous Message | Simon Riggs | 2008-11-03 06:41:25 | Re: Hot standby v5 patch assertion failure |