You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@drill.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2018/08/02 17:17:02 UTC

[jira] [Commented] (DRILL-6645) Transform TopN in Lateral Unnest pipeline to Sort and Limit.

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

ASF GitHub Bot commented on DRILL-6645:
---------------------------------------

HanumathRao opened a new pull request #1417: DRILL-6645: Transform TopN in Lateral Unnest pipeline to Sort and Limit.
URL: https://github.com/apache/drill/pull/1417
 
 
   
   In Lateral/Unnest pipeline, for the sake of getting correct results it is required to introduce a ParitionLimit instead of Limit. ParitionLimit is introduced by PR for DRILL-6652. Similarly a TopN should have another version like PartitionTopN. Since ParitionTopN operator is not yet implemented we can use Limit and sort to replace a TopN. This PR includes changes to transform the TopN -> Sort and Limit.
   
   @gparai  @sohami  Can you please review these changes.
   
   This PR needs to be committed after the DRILL-6652 committed.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


> Transform TopN in Lateral Unnest pipeline to Sort and Limit.
> ------------------------------------------------------------
>
>                 Key: DRILL-6645
>                 URL: https://issues.apache.org/jira/browse/DRILL-6645
>             Project: Apache Drill
>          Issue Type: Improvement
>          Components: Query Planning &amp; Optimization
>    Affects Versions: 1.14.0
>            Reporter: Hanumath Rao Maduri
>            Assignee: Hanumath Rao Maduri
>            Priority: Major
>             Fix For: 1.15.0
>
>
> TopN operator is not supported in Lateral Unnest pipeline. Hence transform the TopN to use Sort and Limit.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)