[ 
https://issues.apache.org/jira/browse/OAK-28?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13234526#comment-13234526
 ] 

Michael Dürig commented on OAK-28:
----------------------------------

Ok agreed. Why do we need to implement a query parser in the jcr bindings 
module then? Isn't passing the query string along with the name space mappings 
and the query language descriptor down to the oak-core sufficient?
                
> Query implementation
> --------------------
>
>                 Key: OAK-28
>                 URL: https://issues.apache.org/jira/browse/OAK-28
>             Project: Jackrabbit Oak
>          Issue Type: New Feature
>          Components: core
>            Reporter: Thomas Mueller
>            Assignee: Thomas Mueller
>         Attachments: OakToJcrQueryTreeConverter.java
>
>
> A query engine needs to be implemented. 
> This includes a query parser in oak-core (where we don't want to use the JCR 
> API), and a query parser in oak-jcr (where the parsed query tree needs to 
> implement the JCR API).
> To avoid writing two independent parsers, I suggest to change the parser to 
> emit a non-JCR query tree (so the parser can be used in oak-core). There 
> needs to be a converter from the non-JCR query tree to a JCR query tree, so 
> the same parser can be used in oak-jcr.
> This will still require two independent query tree implementations (about 37 
> duplicated classes: 37 classes for oak-core and 37 classes in oak-jcr). Plus 
> it requires a tree converter.
> If somebody has a better idea please tell me :-)
> Prototype implementation of the query tree converter. Please note the class 
> names are only to for illustration, but I don't know yet a good naming 
> convention. Ideas are welcome!

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira


Reply via email to