Simplify find_by_statement_cache interaction

...down to a lazily-initialized class-level ivar with a single accessor cached_find_by_statement

As far as I can tell this is the minimal isolation of the feature.
This `||=` isn't thread safe, which is why I did the initialize in the "inherited" hook.