[ 
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)

Reply via email to