Reduce synchronized block scope when loading a node by id

This is a small(ish) change that takes the 'getNodeById' cll outside of the synchronized block that wraps the cache access.
I see the load call as a heavy one, and the proposed change offers better concurrency behavior.

In the tests that I ran, mostly related to queries I got a 20% improvement on average query response time.