Typo in a comment in set_rel_size()

Started by Amit Langoteabout 11 years ago3 messageshackers
Jump to latest
#1Amit Langote
Langote_Amit_f8@lab.ntt.co.jp

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

Attachments:

set_rel_size-comment-typo.patchtext/x-diff; name=set_rel_size-comment-typo.patchDownload+1-1
#2Tom Lane
tgl@sss.pgh.pa.us
In reply to: Amit Langote (#1)
Re: Typo in a comment in set_rel_size()

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

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

#3Amit Langote
Langote_Amit_f8@lab.ntt.co.jp
In reply to: Tom Lane (#2)
Re: Typo in a comment in set_rel_size()

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

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