[jira] [Comment Edited] (HUDI-791) Replace null by Option in Delta Streamer

["Zili Chen (Jira)" <ji...@apache.org>]

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

Zili Chen edited comment on HUDI-791 at 4/16/20, 2:46 PM:
----------------------------------------------------------

I've taken a look at the {{null}} s. It seems most as field or parameter. Actually Java's {{Optional}} is used for return value and it causes serialization issue if as field. Mark as {{@Nullable}} and handle {{null}} s in narrow  places makes more sense to me.


was (Author: tison):
I've taken a look at the {{null}}s. It seems most as field or parameter. Actually Java's {{Optional}} is used for return value and it causes serialization issue if as field. Mark as {{@Nullable}} and handle {{null}}s in narrow  places makes more sense to me.

> Replace null by Option in Delta Streamer
> ----------------------------------------
>
>                 Key: HUDI-791
>                 URL: https://issues.apache.org/jira/browse/HUDI-791
>             Project: Apache Hudi (incubating)
>          Issue Type: Improvement
>          Components: DeltaStreamer, newbie
>            Reporter: Yanjia Gary Li
>            Priority: Minor
>
> There is a lot of null in Delta Streamer. That will be great if we can replace those null by Option. 



--
This message was sent by Atlassian Jira
(v8.3.4#803005)