Re: Change format of FDW options used in \d* commands

From: Shigeru Hanada <shigeru(dot)hanada(at)gmail(dot)com>
To: Robert Haas <robertmhaas(at)gmail(dot)com>
Cc: Alvaro Herrera <alvherre(at)commandprompt(dot)com>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Change format of FDW options used in \d* commands
Date: 2011-08-18 09:43:07
Message-ID: 4E4CDEAB.8070300@gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Thanks for the review.

(2011/08/13 3:59), Robert Haas wrote:
> IMHO, the new format should put parentheses around the options list.

Agreed. Revised version of patch has been attached. This version puts
parentheses around FDW option only when it was not NULL.

Regards,
--
Shigeru Hanada

Attachment Content-Type Size
20110818_fdw_option_format.patch text/plain 46.9 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Dimitri Fontaine 2011-08-18 09:49:44 Re: Displaying accumulated autovacuum cost
Previous Message Ashesh Vashi 2011-08-18 09:21:44 Re: PATCH: Compiling PostgreSQL using ActiveState Python 3.2