You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@samza.apache.org by "Jake Maes (JIRA)" <ji...@apache.org> on 2017/05/10 22:11:04 UTC
[jira] [Resolved] (SAMZA-1277) Add a static merge() operator that
takes all streams to merge as input
[ https://issues.apache.org/jira/browse/SAMZA-1277?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Jake Maes resolved SAMZA-1277.
------------------------------
Resolution: Fixed
Issue resolved by pull request 182
[https://github.com/apache/samza/pull/182]
> Add a static merge() operator that takes all streams to merge as input
> ----------------------------------------------------------------------
>
> Key: SAMZA-1277
> URL: https://issues.apache.org/jira/browse/SAMZA-1277
> Project: Samza
> Issue Type: Bug
> Reporter: Jake Maes
> Assignee: Prateek Maheshwari
> Fix For: 0.13.0
>
>
> The current design forces users to do stuff like this (I can provide more than a few examples already)
> {noFormat}
> List<MessageStream<GenericRecord>> xTrackingMonitoringEvents = new ArrayList<>();
> // ... populate the list
> MessageStream<GenericRecord> initialInputStream =
> xTrackingMonitoringEvents.remove(xTrackingMonitoringEvents.size() - 1);
> Collection<MessageStream<? extends GenericRecord>> otherStreams =
> (Collection<MessageStream<? extends GenericRecord>>) (Object) xTrackingMonitoringEvents;
> initialInputStream
> .merge(otherStreams)
> // .do other stuff
> {noFormat}
> ^ eww. Removing one item from the array so it can be used as the initial message stream seems to cause pain.
> Lets see if we can come up with a cleaner way to do merge()
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)