Re: Foreign table permissions and cloning

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

In response to

Browse pgsql-hackers by date

  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