Re: BUG #5673: Optimizer creates strange execution plan leading to wrong results

On 9/23/2010 9:53 PM, Tom Lane wrote:
> David Schmitt<david@dasz.at>  writes:
>> Executing the attached example.sql on a fresh database demonstrates the
>> problem I'm seeing:
>
> Found it.  If you need a patch right away, it's a one-liner:
>
> diff --git a/src/backend/optimizer/path/allpaths.c b/src/backend/optimizer/path/allpaths.c
> index 2d86da3..b7cf0b8 100644
> --- a/src/backend/optimizer/path/allpaths.c
> +++ b/src/backend/optimizer/path/allpaths.c
> @@ -425,7 +425,7 @@ set_append_rel_pathlist(PlannerInfo *root, RelOptInfo *rel,
>           childpath = childrel->cheapest_total_path;
>           if (IsA(childpath, AppendPath))
>               subpaths = list_concat(subpaths,
> -                                   ((AppendPath *) childpath)->subpaths);
> +                            list_copy(((AppendPath *) childpath)->subpaths));
>           else
>               subpaths = lappend(subpaths, childpath);
>


Thank you very very much, you made my day! *phew*


Best Regards, David
--
dasz.at OG              Tel: +43 (0)664 2602670     Web: http://dasz.at
Klosterneuburg                                         UID: ATU64260999

        FB-Nr.: FN 309285 g          FB-Gericht: LG Korneuburg