Hi Hari-san, On Sunday, February 10, 2019 2:25 PM (GMT+9), Haribabu Kommi wrote: > I try to fix it by adding a check for parallel worker or not and based on it > count them into stats. Patch attached. > > With this patch, currently it doesn't count parallel worker transactions, and > rest of the stats like seqscan and etc are still get counted. IMO they still > may need to be counted as those stats represent the number of tuples > returned and etc. > > Comments?
I took a look at your patch, and it’s pretty straightforward. However, currently the patch does not apply, so I reattached an updated one to keep the CFbot happy. The previous patch also had a missing header to detect parallel workers so I added that. --> #include "access/parallel.h" Regards, Kirk Jamison
0001-Avoid-counting-parallel-worker-transactions-stats_v2.patch
Description: 0001-Avoid-counting-parallel-worker-transactions-stats_v2.patch
