kafka-jira mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Xavier Léauté (JIRA) <j...@apache.org>
Subject [jira] [Commented] (KAFKA-5765) Move merge() from StreamsBuilder to KStream
Date Tue, 22 Aug 2017 17:25:00 GMT

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

Mime
View raw message