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

ASF GitHub Bot commented on FLINK-2861:
---------------------------------------

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

    https://github.com/apache/flink/pull/1387#discussion_r45559955
  
    --- Diff: 
flink-contrib/flink-storm/src/main/java/org/apache/flink/storm/wrappers/WrapperSetupHelper.java
 ---
    @@ -118,6 +119,11 @@ static synchronized TopologyContext 
createTopologyContext(
                        // prefix "Source: " is inserted by Flink sources by 
default -- need to get rid of it here
                        operatorName = operatorName.substring(8);
                }
    +           final int idx = 
operatorName.indexOf(FlinkTopologyBuilder.token);
    +           if (idx != -1) {
    +                   // in case of operator chaining Flink alters the name 
-- need to get rir of it here
    --- End diff --
    
    Typo: "rid"


> Fields grouping on split streams fails
> --------------------------------------
>
>                 Key: FLINK-2861
>                 URL: https://issues.apache.org/jira/browse/FLINK-2861
>             Project: Flink
>          Issue Type: Bug
>          Components: Storm Compatibility
>    Affects Versions: 0.10.0
>            Reporter: Stephan Ewen
>            Assignee: Matthias J. Sax
>
> Using split streams works for shuffle grouping, but not for Fields grouping.
> The reason is that the KeySelector expects an array, and the given type is 
> the {{SplitStreamType}}.



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

Reply via email to