On Fri, Jul 27, 2018 at 11:12 PM, Jonathan S. Katz <jonathan.k...@excoventures.com> wrote: > >> On Jul 27, 2018, at 8:31 AM, Amit Kapila <amit.kapil...@gmail.com> wrote: >> >> >> Yeah, that would be better. Today, I have tried the patch on both >> Head and PG11 and I am getting same and correct results. > > I have applied the the patch to PG11beta2 and tested. >
I think we should backpatch this till 9.6 where the parallel query was introduced. Note, that for HEAD and 11, the patch is same. For PG10, the patch code is same, but because surrounding code is different, the same patch didn't apply. For 9.6, we don't need to collect stats in ExecShutdownNode. I have tested it in all the back branches and it works fine. Robert, We have done verification that the approach works and fixes the bug in all known cases. Do you see any problem with this approach? -- With Regards, Amit Kapila. EnterpriseDB: http://www.enterprisedb.com
fix_gather_stats_PG10_v1.patch
Description: Binary data
fix_gather_stats_PG96_v1.patch
Description: Binary data
fix_gather_stats_PGHead_11_v1.patch
Description: Binary data
