[prev in list] [next in list] [prev in thread] [next in thread] 

List:       postgresql-general
Subject:    Re: [HACKERS] Minor improvement to fdwhandler.sgml
From:       Etsuro Fujita <fujita.etsuro () lab ! ntt ! co ! jp>
Date:       2014-04-30 8:10:50
Message-ID: 5360B00A.2020407 () lab ! ntt ! co ! jp
[Download RAW message or body]

(2014/04/28 23:31), Robert Haas wrote:
> On Thu, Apr 24, 2014 at 7:59 AM, Etsuro Fujita
> <fujita.etsuro@lab.ntt.co.jp> wrote:
>> The patch attached improves docs in fdwhandler.sgml a little bit.
>
> When you submit a patch, it's helpful to describe what the patch
> actually does, rather than just saying it makes things better.  For
> example, I think that this patch could be described as "in
> fdwhandler.sgml, mark references to scan_clauses with <structfield>
> tags".

I thought so.  Sorry, my explanation wasn't enough.

> A problem with that idea is that scan_clauses is not a field in any struct.

I was mistaken.  I think those should be marked with <literal> tags. 
Patch attached.

Thanks,

Best regards,
Etsuro Fujita

["doc-fdw-planning-2.patch" (text/plain)]

diff --git a/doc/src/sgml/fdwhandler.sgml b/doc/src/sgml/fdwhandler.sgml
index 9c818cd..6b5c8b7 100644
--- a/doc/src/sgml/fdwhandler.sgml
+++ b/doc/src/sgml/fdwhandler.sgml
@@ -873,11 +873,11 @@ GetForeignServerByName(const char *name, bool missing_ok);
 
     <para>
      In <function>GetForeignPlan</>, generally the passed-in target list can
-     be copied into the plan node as-is.  The passed scan_clauses list
+     be copied into the plan node as-is.  The passed <literal>scan_clauses</> list
      contains the same clauses as <literal>baserel-&gt;baserestrictinfo</>,
      but may be re-ordered for better execution efficiency.  In simple cases
      the FDW can just strip <structname>RestrictInfo</> nodes from the
-     scan_clauses list (using <function>extract_actual_clauses</>) and put
+     <literal>scan_clauses</> list (using <function>extract_actual_clauses</>) and put
      all the clauses into the plan node's qual list, which means that all the
      clauses will be checked by the executor at run time.  More complex FDWs
      may be able to check some of the clauses internally, in which case those
@@ -895,7 +895,7 @@ GetForeignServerByName(const char *name, bool missing_ok);
      affect the cost estimate for the path.  The path's
      <structfield>fdw_private</> field would probably include a pointer to
      the identified clause's <structname>RestrictInfo</> node.  Then
-     <function>GetForeignPlan</> would remove that clause from scan_clauses,
+     <function>GetForeignPlan</> would remove that clause from <literal>scan_clauses</>,
      but add the <replaceable>sub_expression</> to <structfield>fdw_exprs</>
      to ensure that it gets massaged into executable form.  It would probably
      also put control information into the plan node's


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


[prev in list] [next in list] [prev in thread] [next in thread]