The code did not attempt to build the additional lookup before fetching from
it. Our solution is to switch to requesting the cache in get_????_additional_lookup()
which will trigger the building of the additional lookup.

Committing since we are now beyond the branch point in the API. This
allows a FullIdCache to key on various alternative identifiers/keys
generated by the implementing class. It means that if you want this
functionality you need to sub-class FullIdCache.

After talking to Compara we found the need to support additional
lookups with the full id cache. Since all values are stored in memory
the lookup is very easy to compute.

[ENSCORESW-478] and [ENSCORESW-467]. Brought in a Perl::Critic test being run on the lowest level. Tests are only run when we have the ENV TEST_AUTHOR is set to a true value. Otherwise it does not impact on the current test cases. Also had to fix a number of issues associcated with these flags. Mostly were due to not using warnings or strict. Anything I felt was ok or not my call has been annotated with a no critic flag.

Added a no_cache override feature plus testcase. Useful in times of memory crisis coupled with annoyingly slow lookups of small data.

ENSCORESW-152. Committing the low cardinality and low row count caching framework into the core API. Implement the correct method from BaseAdaptor to have your values cached in local memory