On Fri, Jul 17, 2020 at 5:13 PM Jeff Davis <pg...@j-davis.com> wrote: > The idea is growing on me a bit. It doesn't give exactly v12 behavior, > but it does offer another lever that might tackle a lot of the > practical cases.
Cool. > If I were making the decision alone, I'd still choose > the escape hatch based on simplicity, but I'm fine with this approach > as well. There is also the separate question of what to do about the hashagg_avoid_disk_plan GUC (this is a separate open item that requires a separate resolution). Tom leans slightly towards removing it now. Is your position about the same as before? > The patch itself looks reasonable to me. I don't see a lot of obvious > dangers, but perhaps someone would like to take a closer look at the > planner changes as you suggest. It would be good to get further input on the patch from somebody else, particularly the planner aspects. My intention is to commit the patch myself. I was the primary advocate for hash_mem_multiplier, so it seems as if I should own it. (You may have noticed that I just pushed the preparatory local-variable-renaming patch, to get that piece out of the way.) -- Peter Geoghegan