hi all

recent the ContentSession interface got a new method

 Blob createBlob(InputStream inputStream) throws IOException;

this there is no TODO comment associated with it i assume
that is intended to stay... however, it feels a bit odd to
me having that method on the content session interface and
i don't see how and why a regular API consumer of the
ContentSession would need/use this.

currently the only usage of the method is in a private
method on ValueFactoryImpl, which for that very purpose
needs to have the content session passed to the constructor
but doesn't otherwise need access to the content session
at all...

imo that should be reviewed again.

kind regards
angela



Reply via email to