Content-Length: 260420 | pFad | http://github.com/postgrespro/postgres_cluster/commit/2ae2e9c00798685cd75ea0cc5120466bf2027b90

68 Revert "Remove troublesome Asserts in cost_mergejoin()." · postgrespro/postgres_cluster@2ae2e9c · GitHub
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)








ApplySandwichStrip

pFad - (p)hone/(F)rame/(a)nonymizer/(d)eclutterfier!      Saves Data!


--- a PPN by Garber Painting Akron. With Image Size Reduction included!

Fetched URL: http://github.com/postgrespro/postgres_cluster/commit/2ae2e9c00798685cd75ea0cc5120466bf2027b90

Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy