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

ASF GitHub Bot commented on STORM-1434:
---------------------------------------

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

    https://github.com/apache/storm/pull/1633#discussion_r75264055
  
    --- Diff: 
external/sql/storm-sql-core/src/jvm/org/apache/storm/sql/compiler/ExprCompiler.java
 ---
    @@ -67,8 +68,9 @@ public ExprCompiler(PrintWriter pw, JavaTypeFactory 
typeFactory) {
       public String visitInputRef(RexInputRef rexInputRef) {
         String name = reserveName();
         String typeName = javaTypeName(rexInputRef);
    +    String boxedTypeName = boxedJavaTypeName(rexInputRef);
    --- End diff --
    
    Actually converting Object to primitive type is allowed for Java 7 or 
upper, but Janino throws error while compiling script so I just converted to 
boxed type.


> Support the GROUP BY clause in StormSQL
> ---------------------------------------
>
>                 Key: STORM-1434
>                 URL: https://issues.apache.org/jira/browse/STORM-1434
>             Project: Apache Storm
>          Issue Type: New Feature
>          Components: storm-sql
>            Reporter: Haohui Mai
>
> This jira tracks the effort of implement the support `GROUP BY` clause in 
> StormSQL.



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

Reply via email to