"Albe Laurenz" <laurenz.a...@wien.gv.at> writes:
> 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.

                        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

Reply via email to