Findbugs performance fix: using entrySet instead of keySet

I turned on FindBugs to fail the build on errors, and as a result fixed those errors.  There is only one error category that needed fixing; performance and mostly just one thing:

> <strong>Inefficient use of keySet iterator instead of entrySet iterator</strong>
> This method accesses the value of a Map entry, using a key that was retrieved from a keySet iterator. It is more efficient to use an iterator on the entrySet of the map, to avoid the Map.get(key) lookup.

I will next attempt to look through other categories we might be interested in, such as CORRECTNESS, MT_CORRECTNESS, BAD_PRACTICE and STYLE but we likely have to weed through those to find the ones actually we care about. It is however good to point out that as far as findbugs is concerned, no security issues are currently found. 

You can manually run findbugs via `mvn findbugs:check`. All PRs should do so automatically. 
I could revert, but I am just having a hard time digesting this. `this.services` is not exposed. It's only contained within this class which means nobody else would have a reference to it, except for the unmodifiable map that nobody cant do anything with anyway. I would have understood this better if we actually we exposing services out and naked with no protection. Right?