Tom Lane wrote:
>>> Um, that is what the proposed patch does.

>> I was referring to the first two lines that the patch removes.
>> I guess I don't understand why they should go.

> What we'd have left after the proposed removal is
>
>                            if (new_path->rows < old_path->rows)
>                                  remove_old = true;  /* new dominates old */
>                            else
>                                  accept_new = false; /* old equals or 
> dominates new */
>
> There's no need to make more than one test on the rows values.

I see, thanks for explaining.

Yours,
Laurenz Albe

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

Reply via email to