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

ASF GitHub Bot commented on JENA-1313:
--------------------------------------

Github user ajs6f commented on a diff in the pull request:

    https://github.com/apache/jena/pull/237#discussion_r111237018
  
    --- Diff: jena-arq/src/main/java/org/apache/jena/sparql/expr/NodeValue.java 
---
    @@ -784,10 +786,15 @@ private static int compare(NodeValue nv1, NodeValue 
nv2, boolean sortOrderingCom
                         return x ;
                     }
     
    -                // same lang tag (case insensitive)
    -                x = StrUtils.strCompare(node1.getLiteralLexicalForm(), 
node2.getLiteralLexicalForm()) ;
    +                // same lang tag, handle collation
    +                // TBD: cache locales? cache collators? pre define 
both/any? a simple in-memory lru-map-cache?
    +                Locale desiredLocale = 
Locale.forLanguageTag(node1.getLiteralLanguage());
    +                Collator collator = Collator.getInstance(desiredLocale);
    --- End diff --
    
    Or would we want to collate all literals with the same lang tag together 
and for those with a lang-specific collator available, subsort according 
thereto?


> Language-specific collation in ARQ
> ----------------------------------
>
>                 Key: JENA-1313
>                 URL: https://issues.apache.org/jira/browse/JENA-1313
>             Project: Apache Jena
>          Issue Type: Improvement
>          Components: ARQ
>    Affects Versions: Jena 3.2.0
>            Reporter: Osma Suominen
>
> As [discussed|http://markmail.org/message/v2bvsnsza5ksl2cv] on the users 
> mailing list in October 2016, I would like to change ARQ collation of literal 
> values to be language-aware and respect language-specific collation rules.
> This would probably involve changing at least the 
> [NodeUtils.compareLiteralsBySyntax|https://github.com/apache/jena/blob/master/jena-arq/src/main/java/org/apache/jena/sparql/util/NodeUtils.java#L199]
>  method.
> It currently sorts by lexical value first, then by language tag. Since the 
> collation order needs to be stable across all possible literal values, I 
> think the safest way would be to sort by language tag first, then by lexical 
> value according to the collation rules for that language.
> But what about subtags like {{@en-US}} or {{@pt-BR}}? Can they have different 
> collation rules than the main language? It would be a bit strange if all 
> {{@en-US}} literals sorted after {{@en}} literals...
> It would be good to check how Dydra does this and possibly take the same 
> approach. See the message linked above for further backgound.
> I've been talking with [~kinow] about this and he may be interested in 
> implementing it.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to