Re: file_fdw vs relative paths

Magnus Hagander <magnus@hagander.net>

From: Magnus Hagander <magnus@hagander.net>
To: Ian Barwick <ian.barwick@2ndquadrant.com>
Cc: Bruce Momjian <bruce@momjian.us>, Li Japin <japinli@hotmail.com>, PostgreSQL Developers <pgsql-hackers@lists.postgresql.org>
Date: 2020-09-09T10:42:51Z
Lists: pgsql-hackers
On Wed, Sep 9, 2020 at 3:39 AM Ian Barwick <ian.barwick@2ndquadrant.com>
wrote:

> Hi
>
> On 2020/09/07 2:31, Magnus Hagander wrote:
> > On Mon, Aug 31, 2020 at 5:03 PM Bruce Momjian <bruce@momjian.us <mailto:
> bruce@momjian.us>> wrote:
> >
> >     On Mon, Aug 31, 2020 at 01:16:05PM +0200, Magnus Hagander wrote:
> >      >     Bruce, I've applied and backpatched your docs patch for this.
> >      >
> >      > Gah, and of course right after doing that, I remembered I wanted
> to get a
> >      > second change in :) To solve the "who's this Josh" question, I
> suggest we also
> >      > change the example to point to the data/log directory which is
> likely to exist
> >      > in a lot more of the cases. I keep getting people who ask "who is
> josh" based
> >      > on the /home/josh path. Not that it's that important, but...
> >
> >     Thanks, and agreed.
> >
> >
> > Thanks, applied. I backpacked to 13 but didn't bother with the rest as
> it's not technically *wrong* before..
>
> It's missing the leading single quote from the filename parameter:
>
>      diff --git a/doc/src/sgml/file-fdw.sgml b/doc/src/sgml/file-fdw.sgml
>      (...)
>      -OPTIONS ( filename '/home/josh/data/log/pglog.csv', format 'csv' );
>      +OPTIONS ( filename log/pglog.csv', format 'csv' );
>      (...)
>

GAH.

Thanks!


-- 
 Magnus Hagander
 Me: https://www.hagander.net/ <http://www.hagander.net/>
 Work: https://www.redpill-linpro.com/ <http://www.redpill-linpro.com/>

Commits

  1. Change path in example of file_fdw for logs

  2. Fix docs bug stating file_fdw requires absolute paths

  3. Add contrib/file_fdw foreign-data wrapper for reading files via COPY.