未验证 提交 7fc64d7e 编写于 作者: X xiong-gang 提交者: GitHub

Remove a GPDB_92_MERGE_FIXME

Upstream postgres commit e2fa76d8 extracted `set_base_rel_sizes` out from
`set_base_rel_pathlists`, and path related information doesn't exist in
`set_base_rel_sizes`, as the FIXME indicated,
childrel->cheapest_total_path->rows will not be available.

-           parent_rows += cdbpath_rows(root, childrel->cheapest_total_path);
-           width_avg += cdbpath_rows(root, childrel->cheapest_total_path) * childrel->width;
+           parent_rows += childrel->rows;
+           parent_size += childrel->width * childrel->rows;

Co-authored-by: Alexandra Wang<lewang@pivotal.io>
Co-authored-by: Gang Xiong<gxiong@pivotal.io>
上级 96968046
......@@ -688,10 +688,6 @@ set_append_rel_size(PlannerInfo *root, RelOptInfo *rel,
if (childrel->rows > 0)
{
/*
* GPDB_92_MERGE_FIXME:
* The path for childrel hasn't been created by here.
*/
parent_rows += childrel->rows;
parent_size += childrel->width * childrel->rows;
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册