"Albe Laurenz" <laurenz(dot)albe(at)wien(dot)gv(dot)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