Don't use $reset flag with node_load()

Resetting an entity_load cache bin in Drupal 7 can have more drastic consequences than in previous versions of Drupal, as we can have persistent load caches in place of the PHP static memory cache.

Instead, wherever possible, only flush the necessary entities from the cache, using entity_get_controller($entity_type)->resetCache(array($entity_id));

In this instance we surely only care about whether or not the node we are loading is obtained from the entity load cache; yet if $reset is passed then the entire node cache is purged!

Thankfully no code in menu_node itself calls menu_node_get_node() with the reset flag set, but other modules or custom code using menu_node might be doing so.