| From: | Amit Langote <amitlangote09(at)gmail(dot)com> |
|---|---|
| To: | Robert Haas <robertmhaas(at)gmail(dot)com> |
| Cc: | Amit Langote <Langote_Amit_f8(at)lab(dot)ntt(dot)co(dot)jp>, Etsuro Fujita <fujita(dot)etsuro(at)lab(dot)ntt(dot)co(dot)jp>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: Description of ForeignPath |
| Date: | 2016-04-21 17:39:01 |
| Message-ID: | CA+HiwqFg2uc2jFeDMa6YPP03BxVBWFz+uuWh2h0jKEc5GbFYTQ@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Fri, Apr 22, 2016 at 2:32 AM, Robert Haas <robertmhaas(at)gmail(dot)com> wrote:
> On Wed, Apr 20, 2016 at 3:37 AM, Amit Langote wrote:
>>> I think it'd be better to match the comment with that for
>>> create_foreignscan_path(). So how about "ForeignPath represents a
>>> potential scan of a foreign table, foreign join, or foreign upper-relation
>>> processing"? I think we would probably need to update the comment in
>>> src/backend/optimizer/README (L358), too.
>>
>> That's a lot better. Updated patch attached.
>
> Looks OK to me, too. Committed.
Thanks!
- Amit
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Tom Lane | 2016-04-21 17:48:10 | Re: max_parallel_degree > 0 for 9.6 beta |
| Previous Message | Robert Haas | 2016-04-21 17:35:32 | Re: max_parallel_degree > 0 for 9.6 beta |