On 2017/03/14 10:08, Corey Huinker wrote: >> I don't think the plan itself will change as a result of applying this >> patch. You might however be able to observe some performance improvement. > > I could see no performance improvement, even with 16 separate queries > combined with UNION ALL. Query performance was always with +/- 10% of a 9.6 > instance given the same script. I must be missing something.
Hmm, maybe I'm missing something too.
Anyway, here is an older message on this thread from Horiguchi-san where he shared some of the test cases that this patch improves performance for:
<quote> I measured performance and had the following result.
t0 - SELECT sum(a) FROM <local single table>; pl - SELECT sum(a) FROM <4 local children>; pf0 - SELECT sum(a) FROM <4 foreign children on single connection>; pf1 - SELECT sum(a) FROM <4 foreign children on dedicate connections>;
The result is written as "time<ms> (std dev <ms>)"
IIUC, pf0 and pf1 is the same test case (all 4 foreign tables target the same server) measured with different implementations of the patch.
Thanks, Amit
I reworked the test such that all of the foreign tables inherit from the same parent table, and if you query that you do get async execution. But It doesn't work when just stringing together those foreign tables with UNION ALLs.
I don't know how to proceed with this review if that was a goal of the patch.
