diff options
author | Tom Lane <tgl@sss.pgh.pa.us> | 2013-05-10 17:15:43 -0400 |
---|---|---|
committer | Tom Lane <tgl@sss.pgh.pa.us> | 2013-05-10 17:15:43 -0400 |
commit | 0c2c0f82c2cd6d2fa83955e9d767059639010ffc (patch) | |
tree | 6545d5e69336f87b0a10c41ed031f2689e58204f /src/tutorial/basics.source | |
parent | fd262376e9d81f77bde73ab70fa9abdcbc9c6199 (diff) | |
download | postgresql-0c2c0f82c2cd6d2fa83955e9d767059639010ffc.tar.gz postgresql-0c2c0f82c2cd6d2fa83955e9d767059639010ffc.zip |
Guard against input_rows == 0 in estimate_num_groups().
This case doesn't normally happen, because the planner usually clamps
all row estimates to at least one row; but I found that it can arise
when dealing with relations excluded by constraints. Without a defense,
estimate_num_groups() can return zero, which leads to divisions by zero
inside the planner as well as assertion failures in the executor.
An alternative fix would be to change set_dummy_rel_pathlist() to make
the size estimate for a dummy relation 1 row instead of 0, but that seemed
pretty ugly; and probably someday we'll want to drop the convention that
the minimum rowcount estimate is 1 row.
Back-patch to 8.4, as the problem can be demonstrated that far back.
Diffstat (limited to 'src/tutorial/basics.source')
0 files changed, 0 insertions, 0 deletions