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