Mailing-List: contact dev-help@apex.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@apex.apache.org
From: ilganeli <git@git.apache.org>
To: dev@apex.incubator.apache.org
Reply-To: dev@apex.incubator.apache.org
References: <git-pr-309-apex-malhar@git.apache.org>
In-Reply-To: <git-pr-309-apex-malhar@git.apache.org>
Subject: [GitHub] apex-malhar pull request #309: [APEXMALHAR-2106] Support multiple streams in...
Content-Type: text/plain
Message-Id: <20160711160606.1F797DFFF8@git1-us-west.apache.org>
Date: Mon, 11 Jul 2016 16:06:06 +0000 (UTC)
archived-at: Mon, 11 Jul 2016 16:06:18 -0000

GitHub user ilganeli reopened a pull request:

    https://github.com/apache/apex-malhar/pull/309

    [APEXMALHAR-2106] Support multiple streams in StreamMerger

    * Created a module which creates a binary tree of StreamMerger operators to support merging multiple streams
    * StreamMerger operators are presently allocated as CONTAINER_LOCAL. I would like for them to be THREAD_LOCAL but I was getting the following error when attempting this:
    ```
    Caused by: javax.validation.ValidationException: Locality THREAD_LOCAL invalid for operator OperatorMeta{name=Merger_Tier_0_#_0, operator=StreamMerger{name=null}, attributes={}} with multiple input streams as they origin from different owner OIO operators
    ```
    
    * An example application is included 

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/ilganeli/incubator-apex-malhar APEXMALHAR-2106F

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/apex-malhar/pull/309.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #309
    
----
commit e56d2b00f2114842877ec403b8a60df82b4014a9
Author: Ilya Ganelin <ilya.ganelin@capitalone.com>
Date:   2016-06-02T21:56:39Z

    Created a module to merge multiple streams into a single stream. Created a test application demonstrating this in action. Still need to add more tests.
    
    Fixed error with attempting to use odd number of input streams. Input ports for StreamMerger are now optional.
    
    Dramatically simplified implementation by replacing manual tier creation with a queue of unconnected ports.
    
    Updated locality settings.
    
    Added test to count number of tuples received at output and test equality against all tuples sent across all merged streams.
    
    Added explicit check for values.
    
    Changes to pom.
    
    Restored pom

commit d89df40ffd385c686b67ceef0aefb4f36f6fb5a0
Author: Ilya Ganelin <ilya.ganelin@capitalone.com>
Date:   2016-07-11T15:40:53Z

    Merge remote-tracking branch 'upstream/master' into APEXMALHAR-2106F

commit f654ff6aeb3961cbdc566b266667eadc4d347856
Author: Ilya Ganelin <ilya.ganelin@capitalone.com>
Date:   2016-07-11T15:49:01Z

    Updated to use async execution and fixed comments from PR.

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---