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

ASF GitHub Bot commented on TRAFODION-2270:
-------------------------------------------

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

    https://github.com/apache/incubator-trafodion/pull/795#discussion_r86907546
  
    --- Diff: core/sql/optimizer/OptPhysRelExpr.cpp ---
    @@ -11856,6 +11868,7 @@ SortGroupBy::synthPhysicalProperty(const Context* 
myContext,
                                                                  planNumber,
                                                                  pws);
     
    +      //PhysicalProperty(((isRollup() ||NOT extraOrderExpr().isEmpty()) ? 
ValueIdList() : sppOfChild->getSortKey()),
    --- End diff --
    
    Could you remove this dead code?


> support ORDER BY clause in GROUP_CONCAT function
> ------------------------------------------------
>
>                 Key: TRAFODION-2270
>                 URL: https://issues.apache.org/jira/browse/TRAFODION-2270
>             Project: Apache Trafodion
>          Issue Type: Sub-task
>            Reporter: liu ming
>            Assignee: liu ming
>
> currently, GROUP_CONCAT() in Trafodion cannot recognize order by clause, need 
> to support it.
> for example:
> SELECT student_name,
>           GROUP_CONCAT(DISTINCT test_score
>                     ORDER BY test_score asc SEPARATOR ' ')
>           FROM student
>           GROUP BY student_name;   
>                 



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

Reply via email to