Re: NOT Null constraint on foreign table not working - Mailing list pgsql-hackers

From Rushabh Lathia
Subject Re: NOT Null constraint on foreign table not working
Date
Msg-id CAGPqQf2i+biuPxcaoabA2h-ykL_49ycj28OhwOec9iGNi+zWpA@mail.gmail.com
Whole thread Raw
In response to NOT Null constraint on foreign table not working  (Rushabh Lathia <rushabh.lathia@gmail.com>)
List pgsql-hackers
Please consider attached patch here as earlier attached wrong patch.

Sorry for the inconvenience. 


On Mon, Jan 20, 2014 at 1:21 PM, Rushabh Lathia <rushabh.lathia@gmail.com> wrote:
Hello,

Please consider the following test:

create database foo;
\c foo

create table foo_test ( a int );

\c postgres

create extension if not exists postgres_fdw;
create server foo_server foreign data wrapper postgres_fdw options ( dbname  'foo' );
create user mapping for current_user server foo_server;

create foreign table foo_test ( a int not null) server foo_server;

-- insert should return error for because NOT NULL constraint on column a
postgres=# insert into foo_test values ( null );
INSERT 0 1

postgres=# select * from foo_test;
 a 
---
  
(1 row)

-- clean up
drop foreign table foo_test;
drop server foo_server cascade;
\c postgres
drop database foo;


Analysis:

As per the PG documentation it says that foreign table do support the
NOT NULL, NULL and DEFAULT.


But when I tried the NOT NULL constraint, its not working for the foreign tables.
Looking at the code into ExecInsert(), for the foreign table missed to call
ExecConstraints(). I am not sure whether it is intentional that  we not calling
ExecConstraints() in case of foreign server or its missed. 
Do share your thought on this.

I quickly fix the issue by adding ExecConstraints() call for foreign table and
now test behaving as expected. PFA patch for the same.

Regards, 
Rushabh Lathia




--
Rushabh Lathia
Attachment

pgsql-hackers by date:

Previous
From: Pavel Stehule
Date:
Subject: Re: [HACKERS] using rpmbuild with PostgreSQL 9.2.6 source code
Next
From: Sameer Kumar
Date:
Subject: Re: using rpmbuild with PostgreSQL 9.2.6 source code