[jira] [Commented] (KAFKA-5765) Move merge() from StreamsBuilder to KStream

["Xavier Léauté (JIRA)" <ji...@apache.org>]

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

Xavier Léauté commented on KAFKA-5765:
--------------------------------------

I have a small request when it comes to merge(). The current varargs form generates a lot of compiler warnings that need to be suppressed using {{@SuppressWarnings("unchecked")}}.

Given that the typical merge use-case only involves only a handful of streams, I think it would be useful to provide a couple of overloads that take a fixed number of arguments, similar to what Guave does in [ImmutableList.of(...)|https://google.github.io/guava/releases/21.0/api/docs/com/google/common/collect/ImmutableList.html#of-E-E-E-E-E-E-E-E-E-E-E-]


> Move merge() from StreamsBuilder to KStream
> -------------------------------------------
>
>                 Key: KAFKA-5765
>                 URL: https://issues.apache.org/jira/browse/KAFKA-5765
>             Project: Kafka
>          Issue Type: Bug
>          Components: streams
>    Affects Versions: 0.11.0.0
>            Reporter: Matthias J. Sax
>            Assignee: Matthias J. Sax
>              Labels: needs-kip
>
> Merging multiple {{KStream}} is done via {{StreamsBuilder#merge()}} (formally {{KStreamBuilder#merge()}}). This is quite unnatural and should be done via {{KStream#merge()}}.
> As {{StreamsBuilder}} is not released yet, this is not a backward incompatible change (and KStreamBuilder is already deprecated). We still need a KIP as we add a new method to a public {{KStreams}} API.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)