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

Daniel Kuppitz commented on TINKERPOP-1753:
-------------------------------------------

{quote}
but kinda slow hack
{quote}

Why slow? The impact is the micro- or nano-second range. The usual noise 
between query executions makes the changes unrecognizable.
And why a hack? The inability to compare different number types is rooted in 
the JVM. See:

{noformat}
gremlin> Comparator.naturalOrder().compare(1, 2D)
java.lang.Double cannot be cast to java.lang.Integer
{noformat}

Hence the {{NumberHelper}} extension is a pretty clean solution in my opinion.

> OrderStep not able to order by non-integer numbers
> --------------------------------------------------
>
>                 Key: TINKERPOP-1753
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-1753
>             Project: TinkerPop
>          Issue Type: Bug
>          Components: process
>    Affects Versions: 3.2.5
>            Reporter: Daniel Kuppitz
>            Assignee: Daniel Kuppitz
>
> {{order()}} and {{order(local)}} sometimes fail to sort by non-integer 
> numbers. In some cases it works, but the following pattern (non-integer map 
> values?) seems broken:
> {noformat}
> gremlin> 
> g.V().hasLabel("person").group().by("name").by(outE().values("weight").sum()).order(local).by(values)
> java.lang.Double cannot be cast to java.lang.Integer
> Type ':help' or ':h' for help.
> Display stack trace? [yN]
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to