beam-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Kenneth Knowles (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (BEAM-2166) Remove Coder.Context from the public API
Date Thu, 04 May 2017 02:06:04 GMT

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

Kenneth Knowles commented on BEAM-2166:
---------------------------------------

Big +1

> Remove Coder.Context from the public API
> ----------------------------------------
>
>                 Key: BEAM-2166
>                 URL: https://issues.apache.org/jira/browse/BEAM-2166
>             Project: Beam
>          Issue Type: Improvement
>          Components: sdk-java-core, sdk-py
>    Affects Versions: First stable release
>            Reporter: Robert Bradshaw
>            Assignee: Robert Bradshaw
>             Fix For: First stable release
>
>
> Justification: 
> * Contexts add confusion and complexity to the public API (e.g. https://issues.apache.org/jira/browse/BEAM-1448)
> * Leaf (user-written) coders are nearly always nested.
> * Coders are being removed from sources, which was the initial need for context.
> * It is unclear how much value, if any, this provides for internal performance.
> * There may be performance (as well as simplification) gains in removing this for the
Fn API.
> Fully removing this distinction from the internals can be defered  until the last bullet
points are more completely investigated. 



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

Mime
View raw message