Ian Boston wrote
> Hi,
> Here is an updated patch.
> 
> https://github.com/apache/sling/compare/trunk...ieb:OAK-6575-3-1?expand=1
> 

Thanks for updating the patch.

I think we should move ExternalisableInputStream to o.a.s.resource.api,
so other resource providers could potentially use it as well.

I guess JcrExternalisableInputStream should implement that interface :)

It's usually not necessary to use "immediate=true" on a component. It
prevents lazy instantiation and should only be used if there is a good
reason for it.

Regards

Carsten

-- 
Carsten Ziegeler
Adobe Research Switzerland
cziege...@apache.org

Reply via email to