Non-threadsafe methods in ZooKepperConfigStore

Henry:

""""
Both getConfigs and getLogicalNodeMap return unmodifiable* which read-through to the underlying map. They are therefore prone to concurrent modification exceptions. Both should be defensively copying the maps in question.
""""