On Fri, Mar 11, 2022 at 11:35 PM Amit Langote <amitlangot...@gmail.com> wrote: > Attached is v5, now broken into 3 patches: > > 0001: Some refactoring of runtime pruning code > 0002: Add a plan_tree_walker > 0003: Teach AcquireExecutorLocks to skip locking pruned relations
Repeated the performance tests described in the 1st email of this thread: HEAD: (copied from the 1st email) 32 tps = 20561.776403 (without initial connection time) 64 tps = 12553.131423 (without initial connection time) 128 tps = 13330.365696 (without initial connection time) 256 tps = 8605.723120 (without initial connection time) 512 tps = 4435.951139 (without initial connection time) 1024 tps = 2346.902973 (without initial connection time) 2048 tps = 1334.680971 (without initial connection time) Patched v1: (copied from the 1st email) 32 tps = 27554.156077 (without initial connection time) 64 tps = 27531.161310 (without initial connection time) 128 tps = 27138.305677 (without initial connection time) 256 tps = 25825.467724 (without initial connection time) 512 tps = 19864.386305 (without initial connection time) 1024 tps = 18742.668944 (without initial connection time) 2048 tps = 16312.412704 (without initial connection time) Patched v5: 32 tps = 28204.197738 (without initial connection time) 64 tps = 26795.385318 (without initial connection time) 128 tps = 26387.920550 (without initial connection time) 256 tps = 25601.141556 (without initial connection time) 512 tps = 19911.947502 (without initial connection time) 1024 tps = 20158.692952 (without initial connection time) 2048 tps = 16180.195463 (without initial connection time) Good to see that these rewrites haven't really hurt the numbers much, which makes sense because the rewrites have really been about putting the code in the right place. BTW, these are the numbers for the same benchmark repeated with plan_cache_mode = auto, which causes a custom plan to be chosen for every execution and so unaffected by this patch. 32 tps = 13359.225082 (without initial connection time) 64 tps = 15760.533280 (without initial connection time) 128 tps = 15825.734482 (without initial connection time) 256 tps = 15017.693905 (without initial connection time) 512 tps = 13479.973395 (without initial connection time) 1024 tps = 13200.444397 (without initial connection time) 2048 tps = 12884.645475 (without initial connection time) Comparing them to numbers when using force_generic_plan shows that making the generic plans faster is indeed worthwhile. -- Amit Langote EDB: http://www.enterprisedb.com