On Tue, Mar 6, 2018 at 7:52 PM, Jeevan Chalke
<jeevan.cha...@enterprisedb.com> wrote:

> Changes look good to me and refactoring will be useful for partitionwise
> patches.
> However, will it be good if we add agg_costs into the GroupPathExtraData
> too?
> Also can we pass this to the add_partial_paths_to_grouping_rel() and
> add_paths_to_grouping_rel() to avoid passing can_sort, can_hash and costs
> related details individually to them?

I think so too.

Best Wishes,
Ashutosh Bapat
EnterpriseDB Corporation
The Postgres Database Company

Reply via email to