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

Lines changed: 3 additions & 0 deletions
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)
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy