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

ASF GitHub Bot commented on GEODE-2587:
---------------------------------------

Github user nabarunnag commented on the issue:

    https://github.com/apache/geode/pull/517
  
    @ladyVader I am assuming that there should be tests already present testing 
the functional aspect. One of the tests did fail and caught an issue in my fix.
    @jhuynh1 what do you think?


> Refactor OrderByComparator's compare method
> -------------------------------------------
>
>                 Key: GEODE-2587
>                 URL: https://issues.apache.org/jira/browse/GEODE-2587
>             Project: Geode
>          Issue Type: Bug
>          Components: querying
>            Reporter: nabarun
>
> OrderByComparator's compare method creates a lot of objects / arrays with the 
> intention of comparing two objects.
> But allocation of memory for these objects results in GC to kick in.
> The additional time consumed in GC results in increased execution time for 
> queries with ORDER BY clause.
> This method must be refactored to require less memory allocations in order to 
> compare and thus speed up ORDER BY clause.



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

Reply via email to