[ 
https://issues.apache.org/jira/browse/ODE-296?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12600546#action_12600546
 ] 

Matthieu Riou commented on ODE-296:
-----------------------------------

Patch applied, thanks!

I've actually removed the set of resources on OProcess. Really the OModel 
should be considered immutable after compilation, we shouldn't cache stuff in 
there. And I'm not sure the caching here is not a micro-optimization. Your OS 
will cache the file in memory after it's first read so subsequent reading 
should be really fast. Caching it again in the VM shouldn't make it much faster.

I know, the XSL sheet itself is stored in the OProcess. But this was done when 
we didn't have the process store or a better way to access a filesystem 
resource at runtime, so that should actually be refactored at some point.

Anyway, let me know if your experience is different and we'll find another way 
to cache those resources.

And thanks again for your patch, it's most welcome!

> Support XSLT's document() function
> ----------------------------------
>
>                 Key: ODE-296
>                 URL: https://issues.apache.org/jira/browse/ODE-296
>             Project: ODE
>          Issue Type: New Feature
>          Components: BPEL Runtime
>            Reporter: Ciaran Jessup
>             Fix For: 1.2
>
>         Attachments: AddDocumentSupport.patch, AddDocumentSupportv2.patch, 
> HelloWorld2.zip
>
>
> Currently the BPEL runtime does not support executing XSL stylesheets that 
> use the XSLT document() function to retrieve resources local to the process 
> directory.  

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to