Re: Properly pathify the union planner - Mailing list pgsql-hackers

From Andy Fan
Subject Re: Properly pathify the union planner
Date
Msg-id 87jzn2gmwh.fsf@163.com
Whole thread Raw
In response to Re: Properly pathify the union planner  (David Rowley <dgrowleyml@gmail.com>)
List pgsql-hackers
David Rowley <dgrowleyml@gmail.com> writes:

>>
>> The two if-clauses looks unnecessary, it should be handled by
>> pathkeys_count_contained_in already. The same issue exists in
>> pathkeys_useful_for_ordering as well. Attached patch fix it in master.
>
> I agree.  I'd rather not have those redundant checks in
> pathkeys_useful_for_setop(), and I do want those functions to be as
> similar as possible.  So I think adjusting it in master is a good
> idea.
>
> I've pushed your patch.
>
Thanks for the pushing!


-- 
Best Regards
Andy Fan




pgsql-hackers by date:

Previous
From: Sutou Kouhei
Date:
Subject: Re: Why is pq_begintypsend so slow?
Next
From: Pavel Luzanov
Date:
Subject: Re: Things I don't like about \du's "Attributes" column