From 2fd58096f02777c38edb392f78cb5b4ebd90e9d2 Mon Sep 17 00:00:00 2001 From: Robert Haas Date: Wed, 10 Jan 2018 11:18:40 -0500 Subject: [PATCH] Add missing "return" statement to accumulate_append_subpath. Without this, Parallel Append can end up with extra children. Report by Rajkumar Raghuwanshi. Fix by Amit Khandekar. Brown paper bag bug by me. Discussion: http://postgr.es/m/CAKcux6mBF-NiddyEe9LwymoUC5+wh8bQJ=uk2gGkOE+L8cv=LA@mail.gmail.com --- src/backend/optimizer/path/allpaths.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/backend/optimizer/path/allpaths.c b/src/backend/optimizer/path/allpaths.c index 12a6ee4a22b..c5304b712e6 100644 --- a/src/backend/optimizer/path/allpaths.c +++ b/src/backend/optimizer/path/allpaths.c @@ -1926,6 +1926,7 @@ accumulate_append_subpath(Path *path, List **subpaths, List **special_subpaths) apath->first_partial_path); *special_subpaths = list_concat(*special_subpaths, new_special_subpaths); + return; } } else if (IsA(path, MergeAppendPath)) -- 2.39.5