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

angela edited comment on JCR-3845 at 1/20/15 3:52 PM:
------------------------------------------------------

rev 1653275: commit with minor modification (setting the default back to true 
in order to have backwards compatible behavior).

thanks a lot for the patch!


was (Author: anchela):
commit with minor modification (setting the default back to true in order to 
have backwards compatible behavior).

thanks a lot for the patch!

> Have AuthorizableQueryManager Support Specifying Sort Ignore Case Mode
> ----------------------------------------------------------------------
>
>                 Key: JCR-3845
>                 URL: https://issues.apache.org/jira/browse/JCR-3845
>             Project: Jackrabbit Content Repository
>          Issue Type: Improvement
>          Components: jackrabbit-jcr-commons
>    Affects Versions: 2.8
>            Reporter: Dominique Jäggi
>            Assignee: angela
>             Fix For: 2.9.1
>
>         Attachments: 
> JCR-3845_-_Have_AuthorizableQueryManager_Support_Specifying_Sort_Ignore_Case_Mode.patch
>
>
> Currently the _AuthorizableQueryManager_ hard-sets to ignore the case for any 
> specified sort property, resulting in a query that sorts like this:
> {noformat} ... order by fn:lower-case(@myProperty) {noformat}
> this may lead to performance impact, as the lowercasing cannot be covered by 
> a respective index.
> i therefore suggest to allow users of the _AuthorizableQueryManager_ to 
> specify whether lowercasing is desired or not, as an additional flag 
> _ignoreCase: true | false_ in the _ order | sort:_ section.
> [~anchela], kindly review attached patch.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to