You are viewing a plain text version of this content. The canonical link for it is here.

Posted to jira@kafka.apache.org by "Kvicii.Yu (Jira)" <ji...@apache.org> on 2022/02/03 11:00:02 UTC

[jira] [Commented] (KAFKA-13641) Kafka Streams Scala: Add `Option` to `ValueJoiner` parameters

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

Kvicii.Yu commented on KAFKA-13641:
-----------------------------------

[~yzia2000] 

hi, I didn't understand what you mean, or you can tell me class name#method name need to add Option?

> Kafka Streams Scala: Add `Option` to `ValueJoiner` parameters
> -------------------------------------------------------------
>
>                 Key: KAFKA-13641
>                 URL: https://issues.apache.org/jira/browse/KAFKA-13641
>             Project: Kafka
>          Issue Type: Improvement
>          Components: streams
>            Reporter: Mohammad Yousuf Minhaj Zia
>            Priority: Minor
>
> Since `ValueJoiner` right parameter in `leftJoins`, `outerJoins` can be nullable, I am wondering if can wrap them around Scala `Option`.
> However, there is also the concern that the left hand side value can be null in the case of tombstone messages, in which the `Option` semantics can be misleading. I still feel this could be a useful feature in reducing the number of `NullPointerExceptions`.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)