NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit2ae2e9c

committed

Revert "Remove troublesome Asserts in cost_mergejoin()."

This reverts commitff68b25.The recent change to use -fexcess-precision=standard should make thoseAsserts 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 thecode here.

1 parentdca0a54 commit2ae2e9cCopy full SHA for 2ae2e9c

File tree

1 file changed

-0

lines changed

src/backend/optimizer/path
- costsize.c

1 file changed

-0

lines changed

`‎src/backend/optimizer/path/costsize.c`

Lines changed: 3 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -1925,6 +1925,9 @@ cost_mergejoin(MergePath path, PlannerInfo root, SpecialJoinInfo *sjinfo)`
`1925`	`1925`	`outerendsel=outer_rows /outer_path_rows;`
`1926`	`1926`	`innerendsel=inner_rows /inner_path_rows;`
`1927`	`1927`
	`1928`	`+Assert(outerstartsel <=outerendsel);`
	`1929`	`+Assert(innerstartsel <=innerendsel);`
	`1930`	`+`
`1928`	`1931`	`/* cost of source data */`
`1929`	`1932`
`1930`	`1933`	`if (outersortkeys)/* do we need to sort outer? */`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit2ae2e9c

File tree

1 file changed

1 file changed

`‎src/backend/optimizer/path/costsize.c`

0 commit comments