| From: | Neil Conway <neilc(at)samurai(dot)com> |
|---|---|
| To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
| Cc: | pgsql-hackers(at)postgresql(dot)org |
| Subject: | Re: add_missing_from breaks existing views |
| Date: | 2005-10-25 21:58:52 |
| Message-ID: | 1130277532.8506.87.camel@localhost.localdomain |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Tue, 2005-25-10 at 17:43 -0400, Tom Lane wrote:
> What I suggest we do about this is change addImplicitRTE() to set
> inFromCl true for implicitly added RTEs, so that the view rule will
> later be dumped as if the query had been written per spec.
Sounds reasonable. I wonder if this should be backpatched -- ISTM the
proper representation of the view is with an explicit FROM list anyway,
and making the correctness of view definition dependent on a GUC
variable is only asking for trouble.
> 1. Tell people they may have to set add_missing_from = true to reload
> a dump that contains such views.
pg_dump could itself specify "SET add_missing_from = true" before view
definitions. However, that is definitely a hack :(
-Neil
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Simon Riggs | 2005-10-25 23:13:43 | Re: add_missing_from breaks existing views |
| Previous Message | Tom Lane | 2005-10-25 21:55:21 | Re: expanded \df+ display broken in beta4 |