| From: | Serge Rielau <serge(at)rielau(dot)com> |
|---|---|
| To: | "David G(dot) Johnston" <david(dot)g(dot)johnston(at)gmail(dot)com> |
| Cc: | Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>, David Rowley <david(dot)rowley(at)2ndquadrant(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Craig Ringer <craig(dot)ringer(at)2ndquadrant(dot)com>, Joe Conway <mail(at)joeconway(dot)com>, Mario Becroft <mb(at)true(dot)group>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>, Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, Ilya Shkuratov <motr(dot)ilya(at)ya(dot)ru>, Tomas Vondra <tomas(dot)vondra(at)2ndquadrant(dot)com>, Gavin Flower <GavinFlower(at)archidevsys(dot)co(dot)nz>, Andrew Dunstan <andrew(dot)dunstan(at)2ndquadrant(dot)com> |
| Subject: | Re: CTE inlining |
| Date: | 2017-05-09 19:29:31 |
| Message-ID: | be1f7b5f-0099-4874-8237-df672046f23e@rielau.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Tue, May 9, 2017 at 12:22 PM, David G. Johnston <david(dot)g(dot)johnston(at)gmail(dot)com> wrote:
On Tue, May 9, 2017 at 12:15 PM, Peter Eisentraut < peter(dot)eisentraut(at)2ndquadrant(dot)com [peter(dot)eisentraut(at)2ndquadrant(dot)com] > wrote:
On 5/5/17 08:43, David Rowley wrote:
> How about we get the ball rolling on this in v10 and pull that part
> out of the docs. If anything that'll buy us a bit more wiggle room to
> change this in v11.
>
> I've attached a proposed patch.
If we just tell them that the thing they might have relied on might go
away, without a replacement to suggest, then we're just confusing and
scaring them, no?
We'd end up suggesting our OFFSET 0 hack as true protection. If they know for a fact that their use of CTE for its barrier properties is not supported they are also more likely to document intentional usage with something like: "-- CHANGE THIS ONCE VERSION 11 IS RELEASED!!! --" which would make finding the call sites that need to add the new "MATERIALIZED" keyword much easier. How about adding MATERIALIZED now (in 10) as a noise word. Give people a release to switch over before pulling the rug.. Cheers Serge
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Pavel Stehule | 2017-05-09 19:33:32 | Re: proposal psql \gdesc |
| Previous Message | Tom Lane | 2017-05-09 19:24:21 | Re: idea: custom log_line_prefix components besides application_name |