Re: [HACKERS] File_fdw documentation patch to clarify OPTIONS clause

2013-09-23 Thread Robert Haas
On Fri, Sep 20, 2013 at 4:26 PM, Mike Blackwell mike.blackw...@rrd.com wrote: The file_fdw documentation for the OPTIONS clause references the COPY command's documentation. In the case of COPY, the value for some options (e.g. HEADER, OIDS, ...) is optional. While the file_fdw documentation

Re: [HACKERS] File_fdw documentation patch to clarify OPTIONS clause

2013-09-23 Thread Mike Blackwell
Robert, Thanks for the reply. I have no objections to clarifying the note. Attached is a patch with the text you suggested. Mike​ __ *Mike Blackwell | Technical Analyst, Distribution Services/Rollout Management |

Re: [HACKERS] File_fdw documentation patch to clarify OPTIONS clause

2013-09-23 Thread Robert Haas
On Mon, Sep 23, 2013 at 2:24 PM, Mike Blackwell mike.blackw...@rrd.com wrote: Thanks for the reply. I have no objections to clarifying the note. Attached is a patch with the text you suggested. Committed, thanks. Per relatively-usual practice for doc clarifications, I back-patched this one

[HACKERS] File_fdw documentation patch to clarify OPTIONS clause

2013-09-20 Thread Mike Blackwell
The file_fdw documentation for the OPTIONS clause references the COPY command's documentation. In the case of COPY, the value for some options (e.g. HEADER, OIDS, ...) is optional. While the file_fdw documentation makes no mention of it, the values for all options are required. Attached is a