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

ASF GitHub Bot commented on TAJO-1407:
--------------------------------------

Github user jihoonson commented on the pull request:

    https://github.com/apache/tajo/pull/426#issuecomment-85286436
  
    Oh, I think it's ok. The revised test doesn't contain any comparisons of 
elapsed times.
    In addition, MemSortExec is executed only when there remains a sufficient 
amount of memory.
    Here is my +1, and I'll commit it shortly.
    Thanks for your great work!


> Minor performance improvement of MemSortExec
> --------------------------------------------
>
>                 Key: TAJO-1407
>                 URL: https://issues.apache.org/jira/browse/TAJO-1407
>             Project: Tajo
>          Issue Type: Improvement
>          Components: physical operator
>            Reporter: Navis
>            Assignee: Navis
>            Priority: Minor
>
> Currently, tajo uses Collections.sort() with TupleCompartor, which can incur 
> some overheads on comparing. 



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

Reply via email to