[jira] [Updated] (KAFKA-5228) Revisit Streams DSL JavaDocs

["Ismael Juma (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/KAFKA-5228?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Ismael Juma updated KAFKA-5228:
-------------------------------
    Fix Version/s:     (was: 0.11.0.0)
                   0.11.1.0

> Revisit Streams DSL JavaDocs
> ----------------------------
>
>                 Key: KAFKA-5228
>                 URL: https://issues.apache.org/jira/browse/KAFKA-5228
>             Project: Kafka
>          Issue Type: Improvement
>          Components: streams
>    Affects Versions: 0.10.2.1
>            Reporter: Matthias J. Sax
>            Assignee: Jeyhun Karimov
>            Priority: Trivial
>              Labels: beginner, documentation, newbie
>             Fix For: 0.11.1.0
>
>
> We got some user feedback, that is it sometimes not clear from the JavaDocs, if provides {{Serdes}} are for input or output records.
> For example:
> {noformat}
> ...
>      * @param keySerde key serdes for materializing this stream.
>      *                 If not specified the default serdes defined in the configs will be used
>      * @param valSerde value serdes for materializing this stream,
>      *                 if not specified the default serdes defined in the configs will be used
> ...
>     <VT, VR> KStream<K, VR> join(final KTable<K, VT> table,
>                                  final ValueJoiner<? super V, ? super VT, ? extends VR> joiner,
>                                  final Serde<K> keySerde,
>                                  final Serde<V> valSerde);
> {noformat}
> The phrase "for this stream" means the input stream. But it is rather subtle. We should revisit the complete JavaDocs and rephrase the Serde parameter description if required. We should also rename the parameter names (in the example about, maybe from {{keySerde}} to {{inputKStreamKeySerde}})



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)