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

ASF GitHub Bot commented on APEXMALHAR-2106:
--------------------------------------------

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

    
https://github.com/apache/incubator-apex-malhar/pull/307#discussion_r65472797
  
    --- Diff: 
library/src/main/java/com/datatorrent/lib/stream/StreamMerger.java ---
    @@ -41,35 +47,57 @@
     @Stateless
     public class StreamMerger<K> extends BaseOperator
     {
    -  /**
    -   * Data input port 1.
    -   */
    -  public final transient DefaultInputPort<K> data1 = new 
DefaultInputPort<K>()
    +  private int portCount = 0;
    +  private final transient ArrayList<DefaultInputPort<K>> ports = new 
ArrayList<>(portCount);
    --- End diff --
    
    Did you use this operator in a DAG and see if it works?


> Support merging multiple streams with StreamMerger 
> ---------------------------------------------------
>
>                 Key: APEXMALHAR-2106
>                 URL: https://issues.apache.org/jira/browse/APEXMALHAR-2106
>             Project: Apache Apex Malhar
>          Issue Type: New Feature
>            Reporter: Ilya Ganelin
>            Assignee: Ilya Ganelin
>
> To properly implement the Flatten transformation (and other Stream 
> combination operations), Apex must support merging data from multiple 
> sources. The StreamMerger operator can be improved to merge multiple streams, 
> rather than just the two streams it can handle in the present implementation. 



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

Reply via email to