Added javadoc

Hopefully that's a good start... 
Depends I guess, I would have imagined that returning the `<T>` would work, but possibly returning the `ValueHolder.value()` could have been failing. But maybe that's tying the 2 classes to tightly (arguably again, they're both `Store` "subtypes")