On Tue, Mar 14, 2017 at 9:59 PM, Robert Haas <[email protected]> wrote: > On Tue, Jan 17, 2017 at 11:49 AM, Robert Haas <[email protected]> wrote: >> On Mon, Jan 16, 2017 at 7:23 AM, Amit Kapila <[email protected]> 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 ([email protected]) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
