Massive performance improvement on correctBlockNodeIds
authorLoic Blot <loic.blot@unix-experience.fr>
Wed, 26 Jul 2017 20:54:55 +0000 (22:54 +0200)
committerLoïc Blot <nerzhul@users.noreply.github.com>
Thu, 27 Jul 2017 05:56:48 +0000 (07:56 +0200)
commit61e487719017511fdc37a944ea7321da46d28ee4
treef922dbc5428ab19b7966b0d6105432ab43076067
parentbc1654feedc90caa8c26328ca6f0fc59fbe5b76c
Massive performance improvement on correctBlockNodeIds

correctBlockNodeIds does 2 lookups for each loaded node, one to translate DB ID to name and a second to translate name to real ID. Name to real ID is very consumming if done on every node. As mapblocks are in most cases composed of many identical adjacent nodes, cache previous source and destination id and use them on the next node to prevent any lookup on those maps.

This reduce the function load from 15% of my CPU usage to ~0.7%, on the test, calls was reduced from 2.5M lookups to 42k lookups, it's a huge performance gain
src/mapblock.cpp