postgres.git / summary / log / commit / refs

commit    2fd58096f02777c38edb392f78cb5b4ebd90e9d2
Author:   Robert Haas <rhaas@postgresql.org>
Date:     Wed Jan 10 16:18:40 2018 +0000

    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))


[parent: b3617cdfbba1]