Whenever we do a convert, this function is run now. This means the full kb is built everytime. I made a minor optimization for aps_harvest here: https://github.com/inspirehep/harvesting-kit/commit/9747c4114eef75d5cadbc96501700b30bd3ee5be#diff-b85c2d89d7262354900ca80618018254R40

This allows to run it only once when executing if passed to the init function. We could do it smarter here and use something like memoize: https://wiki.python.org/moin/PythonDecoratorLibrary#Memoize.

Also this code is the same in many of the packages we have so far, perhaps it can be kept in some superclass with all the similar functions if you think that make sense.