You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by "Till Rohrmann (JIRA)" <ji...@apache.org> on 2019/06/13 15:37:00 UTC

[jira] [Resolved] (FLINK-12822) Add explicit transformer from SerializableOptional to Optional

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

Till Rohrmann resolved FLINK-12822.
-----------------------------------
    Resolution: Done

Done via 27a1c415d39d07790c322fe12d7c4e8075477cea

> Add explicit transformer from SerializableOptional to Optional
> --------------------------------------------------------------
>
>                 Key: FLINK-12822
>                 URL: https://issues.apache.org/jira/browse/FLINK-12822
>             Project: Flink
>          Issue Type: Improvement
>          Components: API / Type Serialization System
>    Affects Versions: 1.9.0
>            Reporter: TisonKun
>            Assignee: TisonKun
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 1.9.0
>
>         Attachments: FLINK-12822.patch
>
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> We introduce {{SerializableOptional}} to represent return value that of {{Optional}} and transported cross network.
> The purpose is reasonable. However, a wart is inside {{SerializableOptional}}. Calling {{SerializableOptional#map}} will returns a {{Optional}}, which might surprise contributors if they want to chain operations and get a {{SerializableOptional}} as final result(to another transport maybe).
> Semantically return value of {{SerializableOptional#map}} should be {{SerializableOptional}}, and for interoperation, we could introduce a {{SerializableOptional#toOptional}} which easily adapt to the correct type.
> cc the original author [~till.rohrmann]



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)