Skip to content

Commit 2ae2e9c

Browse files
committed
Revert "Remove troublesome Asserts in cost_mergejoin()."
This reverts commit ff68b25. The recent change to use -fexcess-precision=standard should make those Asserts safe, and does fix a test case that formerly crashed for me, so I think there's no need to have a cross-version difference in the code here.
1 parent dca0a54 commit 2ae2e9c

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

src/backend/optimizer/path/costsize.c

+3
Original file line numberDiff line numberDiff line change
@@ -1925,6 +1925,9 @@ cost_mergejoin(MergePath *path, PlannerInfo *root, SpecialJoinInfo *sjinfo)
19251925
outerendsel = outer_rows / outer_path_rows;
19261926
innerendsel = inner_rows / inner_path_rows;
19271927

1928+
Assert(outerstartsel <= outerendsel);
1929+
Assert(innerstartsel <= innerendsel);
1930+
19281931
/* cost of source data */
19291932

19301933
if (outersortkeys) /* do we need to sort outer? */

0 commit comments

Comments
 (0)