Re: SQL/MED - file_fdw - Mailing list pgsql-hackers

From Andrew Dunstan
Subject Re: SQL/MED - file_fdw
Date
Msg-id 4CFABFA4.9080500@dunslane.net
Whole thread Raw
In response to SQL/MED - file_fdw  (Shigeru HANADA <hanada@metrosystems.co.jp>)
Responses Re: SQL/MED - file_fdw
List pgsql-hackers

On 11/25/2010 03:12 AM, Shigeru HANADA wrote:
> Hi, hackers,
>
> Attached is a patch that adds file_fdw, FDW which reads records from
> files on the server side, as a contrib module.  This patch is based on
> "SQL/MED core functionality" patch.
>
> [SQL/MED - core functionality]
> http://archives.postgresql.org/pgsql-hackers/2010-11/msg01698.php
>
> File_fdw can be installed with the steps similar to other contrib
> modules, and you can create FDW with the script:
>      $SHAREDIR/contrib/file_fdw.sql
> Note that you need to create file_fdw for each database.
>
> Document for file_fdw is included in the patch, although the contents
> might not be enough.
>
> Any comments and questions are welcome.


Looking at file_parser.c, it seems to be largely taken from copy.c. 
Wouldn't it be better to call those functions, or refactor them so they 
are callable if necessary?

cheers

andrew





pgsql-hackers by date:

Previous
From: Josh Berkus
Date:
Subject: Re: Patch to add a primary key using an existing index
Next
From: Robert Haas
Date:
Subject: Re: FK's to refer to rows in inheritance child