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
