Thread: Typo in a comment in set_rel_size()

Hi,

Attached fixes what I suppose is a typo:

         * so set up a single dummy path for it.  Here we only check this for
         * regular baserels; if it's an otherrel, CE was already checked in
-        * set_append_rel_pathlist().
+        * set_append_rel_size().
         *

Thanks,
Amit

Amit Langote <Langote_Amit_f8@lab.ntt.co.jp> writes:
> Attached fixes what I suppose is a typo:

>          * so set up a single dummy path for it.  Here we only check this for
>          * regular baserels; if it's an otherrel, CE was already checked in
> -        * set_append_rel_pathlist().
> +        * set_append_rel_size().
>          *

It's not a typo; the comment was correct when written.  But I evidently
missed updating it when set_append_rel_pathlist() got split into two
functions.  Applied, thanks for noticing!
        regards, tom lane

On 2015-04-25 AM 04:20, Tom Lane wrote:          *
> 
> It's not a typo; the comment was correct when written.  But I evidently
> missed updating it when set_append_rel_pathlist() got split into two
> functions.  Applied, thanks for noticing!
> 

Ah, thanks!

Amit