On Tue, Mar 14, 2017 at 9:59 PM, Robert Haas <robertmh...@gmail.com> wrote: > On Tue, Jan 17, 2017 at 11:49 AM, Robert Haas <robertmh...@gmail.com> wrote: >> On Mon, Jan 16, 2017 at 7:23 AM, Amit Kapila <amit.kapil...@gmail.com> wrote: >>> >>> >>> Isn't it better to call clamp_row_est in join costing functions as we >>> are doing in cost_seqscan()? Is there a reason to keep those >>> different? >> >> No, those should probably be changed to match. > > So I guess that'd look something like this? >
Yes, the patch looks good to me. -- With Regards, Amit Kapila. EnterpriseDB: http://www.enterprisedb.com -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers