diff options
| author | Robert Haas <rhaas@postgresql.org> | 2018-01-10 11:18:40 -0500 |
|---|---|---|
| committer | Robert Haas <rhaas@postgresql.org> | 2018-01-10 11:21:20 -0500 |
| commit | 2fd58096f02777c38edb392f78cb5b4ebd90e9d2 (patch) | |
| tree | 1895060c48942ef860c6bb2f032cd87ac2dd90c7 /src | |
| parent | b3617cdfbba1b5381e9d1c6bc0839500e8eb7273 (diff) | |
| download | postgresql-2fd58096f02777c38edb392f78cb5b4ebd90e9d2.tar.gz postgresql-2fd58096f02777c38edb392f78cb5b4ebd90e9d2.zip | |
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
Diffstat (limited to 'src')
| -rw-r--r-- | src/backend/optimizer/path/allpaths.c | 1 |
1 files changed, 1 insertions, 0 deletions
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)) |