Is this `all_cached_associations_needing_population` method ever appropriate?  It looks like it is being used to populate association caches on cache misses, which we shouldn't need to do for non-embedded cached associations.  I'll investigate and create another pull request if appropriate.