aboutsummaryrefslogtreecommitdiff
path: root/doc/src
diff options
context:
space:
mode:
authorTom Lane <tgl@sss.pgh.pa.us>2013-05-10 17:15:30 -0400
committerTom Lane <tgl@sss.pgh.pa.us>2013-05-10 17:15:30 -0400
commit69cc60dcfd0fb643cd2fe3ce66d4389858bfdeb5 (patch)
treee6221e30c5f480aeef3b3640a7a1dcfaea76a69b /doc/src
parent477b5a0e24f3b62a470f9684e22e36a2c7735274 (diff)
downloadpostgresql-69cc60dcfd0fb643cd2fe3ce66d4389858bfdeb5.tar.gz
postgresql-69cc60dcfd0fb643cd2fe3ce66d4389858bfdeb5.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 'doc/src')
0 files changed, 0 insertions, 0 deletions