| From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
|---|---|
| To: | Vik Reykja <vikreykja(at)gmail(dot)com> |
| Cc: | PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: 9.2rc1 produces incorrect results |
| Date: | 2012-09-05 04:09:03 |
| Message-ID: | 3521.1346818143@sss.pgh.pa.us |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
I wrote:
> I think probably the best fix is to rejigger things so that Params
> assigned by different executions of SS_replace_correlation_vars and
> createplan.c can't share PARAM_EXEC numbers. This will result in
> rather larger ecxt_param_exec_vals arrays at runtime, but the array
> entries aren't very large, so I don't think it'll matter.
Attached is a draft patch against HEAD for this. I think it makes the
planner's handling of outer-level Params far less squishy than it's ever
been, but it is rather a large change. Not sure whether to risk pushing
it into 9.2 right now, or wait till after we cut 9.2.0 ... thoughts?
regards, tom lane
| Attachment | Content-Type | Size |
|---|---|---|
| planner-param-handling-1.patch | text/x-patch | 47.5 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Robert Haas | 2012-09-05 04:17:06 | Re: too much pgbench init output |
| Previous Message | Bruce Momjian | 2012-09-05 04:02:24 | Re: 9.2 pg_upgrade regression tests on WIndows |