From: | Thom Brown <thom(at)linux(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Robert Haas <robertmhaas(at)gmail(dot)com>, Shigeru HANADA <hanada(at)metrosystems(dot)co(dot)jp>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Foreign table permissions and cloning |
Date: | 2011-04-15 09:48:32 |
Message-ID: | BANLkTikaAvy16afdN=ngUp9BLt8ykVCKyg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 15 April 2011 04:26, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> Why is this a documentation issue and not a code issue? IMO we should
> flat out reject both NOT NULL and DEFAULT declarations on foreign
> tables, until such time as we're prepared to do something useful with
> them. Reasons:
If the removal the redundant declarations is do-able for this release,
that would be preferable. And if that change is to happen, I guess it
has to start happening immediately, letting the pgAdmin guys know too.
> 1. Accepting non-functional constraint declarations is something we've
> been heard to ridicule mysql for. With good reason.
Fair point.
> 2. It probably won't be too long before the planner makes optimization
> decisions that assume NOT NULL declarations to be truthful. When that
> day comes, I don't want to be seeing an exception for foreign tables in
> that logic.
Makes sense.
> 3. When we do get around to making it actually work, we will have a
> backwards-compatibility problem if prior versions accepted the
> declaration but treated it as a no-op.
Probably the most important point.
--
Thom Brown
Twitter: @darkixion
IRC (freenode): dark_ixion
Registered Linux user: #516935
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Radosław Smogura | 2011-04-15 10:32:37 | MMAP Buffers |
Previous Message | Dimitri Fontaine | 2011-04-15 08:32:52 | Re: WIP: Allow SQL-language functions to reference parameters by parameter name |