Lists: | pgsql-hackers |
---|
From: | pasman pasmański <pasman(dot)p(at)gmail(dot)com> |
---|---|
To: | pgsql-hackers(at)postgresql(dot)org |
Subject: | Extend file_fdw wrapper |
Date: | 2011-10-10 07:08:42 |
Message-ID: | CAOWY8=aAknnXiZ+51izarhxR7fM++HCmXjdQ-nQPHBz=0eL7eg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
Attached patch.
------------
pasman
Attachment | Content-Type | Size |
---|---|---|
0001-Extend-file_fdw-wrapper.patch | application/octet-stream | 1.3 KB |
From: | Thom Brown <thom(at)linux(dot)com> |
---|---|
To: | pasman pasmański <pasman(dot)p(at)gmail(dot)com> |
Cc: | pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: Extend file_fdw wrapper |
Date: | 2011-10-10 07:24:59 |
Message-ID: | CAA-aLv4Hj+m+VwaMmvnL9CjqsJPKWmFw4BuFYvYnM5-45tjhvQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
2011/10/10 pasman pasmański <pasman(dot)p(at)gmail(dot)com>:
> Attached patch.
... and what are these new options intended to do?
--
Thom Brown
Twitter: @darkixion
IRC (freenode): dark_ixion
Registered Linux user: #516935
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From: | Shigeru Hanada <shigeru(dot)hanada(at)gmail(dot)com> |
---|---|
To: | pasman pasmański <pasman(dot)p(at)gmail(dot)com> |
Cc: | pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: Extend file_fdw wrapper |
Date: | 2011-10-10 13:51:03 |
Message-ID: | CAEZqfEc1YVFnoGtrBz=OD73pgvSnAE9_5rKTDsi523vJfs917A@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
At a quick glance, this patch seems to have an issue about priority.
Which value is used if an option has been set both on a foreign table
and a foreign server?
Also I think documents and regression tests would be required for
this kind of change.
Regards,
--
Shigeru Hanada
From: | David Fetter <david(at)fetter(dot)org> |
---|---|
To: | Shigeru Hanada <shigeru(dot)hanada(at)gmail(dot)com> |
Cc: | pasman pasmański <pasman(dot)p(at)gmail(dot)com>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: Extend file_fdw wrapper |
Date: | 2011-10-10 13:57:39 |
Message-ID: | 20111010135739.GA20190@fetter.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
On Mon, Oct 10, 2011 at 10:51:03PM +0900, Shigeru Hanada wrote:
> At a quick glance, this patch seems to have an issue about priority.
> Which value is used if an option has been set both on a foreign table
> and a foreign server?
I believe that the finer-grained setting should always override the
coarser, so in this case, the setting for the table should take
precedence over the setting for the server.
> Also I think documents and regression tests would be required for
> this kind of change.
+1 :)
Cheers,
David.
--
David Fetter <david(at)fetter(dot)org> http://fetter.org/
Phone: +1 415 235 3778 AIM: dfetter666 Yahoo!: dfetter
Skype: davidfetter XMPP: david(dot)fetter(at)gmail(dot)com
iCal: webcal://www.tripit.com/feed/ical/people/david74/tripit.ics
Remember to vote!
Consider donating to Postgres: http://www.postgresql.org/about/donate
From: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
---|---|
To: | Shigeru Hanada <shigeru(dot)hanada(at)gmail(dot)com> |
Cc: | pasman pasmański <pasman(dot)p(at)gmail(dot)com>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: Extend file_fdw wrapper |
Date: | 2011-10-10 14:23:51 |
Message-ID: | 4E92FFF7.3020607@dunslane.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
On 10/10/2011 09:51 AM, Shigeru Hanada wrote:
> At a quick glance, this patch seems to have an issue about priority.
> Which value is used if an option has been set both on a foreign table
> and a foreign server?
>
> Also I think documents and regression tests would be required for
> this kind of change.
>
>
I'm not even sure I understand why we should want this anyway. The
closest analog I can think of to a more conventional server is that the
whole file system is the foreign server, and there just don't seem to be
any relevant options at that level. All the options being supplied seem
much saner left as just foreign table options.
cheers
andrew
From: | David Fetter <david(at)fetter(dot)org> |
---|---|
To: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
Cc: | Shigeru Hanada <shigeru(dot)hanada(at)gmail(dot)com>, pasman pasmański <pasman(dot)p(at)gmail(dot)com>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: Extend file_fdw wrapper |
Date: | 2011-10-10 15:59:28 |
Message-ID: | 20111010155928.GA22980@fetter.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
On Mon, Oct 10, 2011 at 10:23:51AM -0400, Andrew Dunstan wrote:
> On 10/10/2011 09:51 AM, Shigeru Hanada wrote:
> >At a quick glance, this patch seems to have an issue about priority.
> >Which value is used if an option has been set both on a foreign table
> >and a foreign server?
> >
> >Also I think documents and regression tests would be required for
> >this kind of change.
>
> I'm not even sure I understand why we should want this anyway. The
> closest analog I can think of to a more conventional server is that
> the whole file system is the foreign server, and there just don't
> seem to be any relevant options at that level. All the options being
> supplied seem much saner left as just foreign table options.
You raise an excellent point, which is that there probably should be
options at that level which override the (settable) generic file_fdw
options.
Cheers,
David.
--
David Fetter <david(at)fetter(dot)org> http://fetter.org/
Phone: +1 415 235 3778 AIM: dfetter666 Yahoo!: dfetter
Skype: davidfetter XMPP: david(dot)fetter(at)gmail(dot)com
iCal: webcal://www.tripit.com/feed/ical/people/david74/tripit.ics
Remember to vote!
Consider donating to Postgres: http://www.postgresql.org/about/donate
From: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
---|---|
To: | David Fetter <david(at)fetter(dot)org> |
Cc: | Shigeru Hanada <shigeru(dot)hanada(at)gmail(dot)com>, pasman pasmański <pasman(dot)p(at)gmail(dot)com>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: Extend file_fdw wrapper |
Date: | 2011-10-10 16:04:30 |
Message-ID: | 4E93178E.6030301@dunslane.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
On 10/10/2011 11:59 AM, David Fetter wrote:
> On Mon, Oct 10, 2011 at 10:23:51AM -0400, Andrew Dunstan wrote:
>> On 10/10/2011 09:51 AM, Shigeru Hanada wrote:
>>> At a quick glance, this patch seems to have an issue about priority.
>>> Which value is used if an option has been set both on a foreign table
>>> and a foreign server?
>>>
>>> Also I think documents and regression tests would be required for
>>> this kind of change.
>> I'm not even sure I understand why we should want this anyway. The
>> closest analog I can think of to a more conventional server is that
>> the whole file system is the foreign server, and there just don't
>> seem to be any relevant options at that level. All the options being
>> supplied seem much saner left as just foreign table options.
> You raise an excellent point, which is that there probably should be
> options at that level which override the (settable) generic file_fdw
> options.
>
>
That's not my point at all.
cheers
andrew