| Lists: | pgsql-hackers |
|---|
| From: | Neil Conway <neilc(at)samurai(dot)com> |
|---|---|
| To: | pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | add_missing_from in 8.1 |
| Date: | 2005-03-31 01:27:16 |
| Message-ID: | 424B51F4.1060708@samurai.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Lists: | pgsql-hackers |
I'd like to make add_missing_from=false the default for 8.1. Any objections?
I think the only reason not to do this is for backward compatibility
with previous releases of PostgreSQL, but I don't find that too
compelling: it is easy to fix up queries that rely on the old behavior,
and the old behavior if not compliant with the SQL spec. Also, the admin
can always enable add_missing_from locally if they can't afford to
update their applications. Per usual, we should mention the change
prominently in the "Incompatibilities" section of the release notes.
Barring any objections, I'll send in a patch that implements this soon.
-Neil
| From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
|---|---|
| To: | Neil Conway <neilc(at)samurai(dot)com> |
| Cc: | pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: add_missing_from in 8.1 |
| Date: | 2005-03-31 01:48:20 |
| Message-ID: | 877.1112233700@sss.pgh.pa.us |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Lists: | pgsql-hackers |
Neil Conway <neilc(at)samurai(dot)com> writes:
> I'd like to make add_missing_from=false the default for 8.1. Any objections?
We still don't have a solution for DELETE, do we?
regards, tom lane
| From: | Neil Conway <neilc(at)samurai(dot)com> |
|---|---|
| To: | Neil Conway <neilc(at)samurai(dot)com> |
| Cc: | pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: add_missing_from in 8.1 |
| Date: | 2005-04-08 01:13:46 |
| Message-ID: | 4255DACA.1030205@samurai.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Lists: | pgsql-hackers |
Neil Conway wrote:
> I'd like to make add_missing_from=false the default for 8.1. Any
> objections?
FYI, I've applied a patch that makes this change to CVS HEAD.
-Neil