On Mon, Jul 27, 2020 at 5:10 PM Jeff Davis <pgsql@j-davis.com> wrote:
> I noticed that one of the conditionals, "cheapest_total_path != NULL",
> was already redundant with the outer conditional before your patch. I
> guess that was just a mistake which your patch corrects along the way?
Makes sense.
> Anyway, the patch looks good to me. We can have a separate discussion
> about pessimizing the costing, if necessary.
Pushed the hashagg_avoid_disk_plan patch -- thanks!
--
Peter Geoghegan