You are viewing a plain text version of this content. The canonical link for it is here.
Posted to oak-issues@jackrabbit.apache.org by "Davide Giannella (JIRA)" <ji...@apache.org> on 2014/08/11 14:51:12 UTC

[jira] [Comment Edited] (OAK-1763) OrderedIndex does not comply with JCR's compareTo semantics

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

Davide Giannella edited comment on OAK-1763 at 8/11/14 12:50 PM:
-----------------------------------------------------------------

As said in [oak-dev|http://markmail.org/message/k363be3q7bodz6jn] even the date encoding is affected even if marginally as the timezone will rarely change on the server side.

Nevertheless a proposed solution for encoding of dates is something like the following

https://gist.github.com/davidegiannella/e278a8178f151bef1150

*edit*. As the encoding applies to longs it could be reused for numbers as well.


was (Author: edivad):
As said in [oak-dev|http://markmail.org/message/k363be3q7bodz6jn] even the date encoding is affected even if marginally as the timezone will rarely change on the server side.

Nevertheless a proposed solution for encoding of dates is something like the following

https://gist.github.com/davidegiannella/e278a8178f151bef1150



> OrderedIndex does not comply with JCR's compareTo semantics
> -----------------------------------------------------------
>
>                 Key: OAK-1763
>                 URL: https://issues.apache.org/jira/browse/OAK-1763
>             Project: Jackrabbit Oak
>          Issue Type: Bug
>          Components: core
>            Reporter: Michael Dürig
>
> The ordered index currently uses the lexicographical order of the string representation of the values. This does not comply with [JCR's compareTo sementics | http://www.day.com/specs/jcr/2.0/3_Repository_Model.html#3.6.5.1%20CompareTo%20Semantics] for e.g. double values. 



--
This message was sent by Atlassian JIRA
(v6.2#6252)